Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UPS welder welding but not consuming charges (infinite battery) #45330

Closed
Xpyder opened this issue Nov 10, 2020 · 7 comments · Fixed by #45781
Closed

UPS welder welding but not consuming charges (infinite battery) #45330

Xpyder opened this issue Nov 10, 2020 · 7 comments · Fixed by #45781
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling <Exploit> Unintended interactions or behavior, usually breaking how the game is balanced (S2 - Confirmed) Bug that's been confirmed to exist
Milestone

Comments

@Xpyder
Copy link
Contributor

Xpyder commented Nov 10, 2020

Describe the bug

Welder modified for UPS doesn't consume charges unless both are in inventory

Steps To Reproduce

Setup
Modified Makeshift Welder to use UPS
Modified UPS (taken from disassembling electric jackhammer) to accept small storage batteries

Tests:

+ welder and ups in inventory - can use, Charges consumed
- Welder nearby and UPS in inventory - can use, no charges consumed
- Welder in inventory and UPS nearby - can use, no charges consumed
- welder and UPS both nearby - can use, no charges consumed
+ welder nearby and UPS far away - can't use, not enough charges

Expected behavior

The algorithm appears to correctly identify when a UPS is nearby that has charges it could consume
I would expect it to consume those charges when used

Screenshots

Ask if needed

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1909
  • Game Version: 0.E-7424-gcbcf5fd [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Magiclysm [magiclysm],
    Alternative Map Key [alt_map_key],
    Mythical Martial Arts [MMA],
    Stats Through Kills [stats_through_kills]
    ]

Additional context

Example save attached for easy testing
Mont Clare.zip

@Xpyder Xpyder mentioned this issue Nov 10, 2020
14 tasks
@PatrikLundell
Copy link
Contributor

I'd suspect it's the same issue as #45018

@Xpyder
Copy link
Contributor Author

Xpyder commented Nov 11, 2020

Interesting, I assumed it was something in the UPS patch that came through so I only searched for UPS issues. But it's certainly possible.

fwiw this also happens with ups hotplates when cooking so it's not just welders.

@wapcaplet wapcaplet added the <Exploit> Unintended interactions or behavior, usually breaking how the game is balanced label Nov 11, 2020
@wapcaplet
Copy link
Contributor

wapcaplet commented Nov 11, 2020

Confirmed in 0.E-7424-gcbcf5fd with a UPS-modded hotplate (and powered UPS on the ground nearby). I can cook food with the hotplate without consuming any UPS charges.

Note, the hotplate only appears usable from the crafting menu. I cannot directly activate the hotplate to heat anything, since the UPS power is not in my inventory.

@wapcaplet wapcaplet added (S2 - Confirmed) Bug that's been confirmed to exist <Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling labels Nov 11, 2020
@PaulBlay
Copy link

Not checked in latest experimental, but UPS electric jackhammer does not use any charges to (a)ctivate in experimental 11142.

@RoyBerube
Copy link
Contributor

Some unexpected behavior happens when crafting with a UPS tool. In this case an arc welder.

  • UPS in inventory and arc welder on ground: crafting fails at 4% with message "You no longer have a none and can't continue crafting"

  • UPS and arc welder both on the ground in the same location: craft completes but without consuming any UPS charge.

@RoyBerube
Copy link
Contributor

RoyBerube commented Nov 30, 2020

I'm close to making a PR for this. The major issues are solved. Needs a little more work on some minor problems.

@Raikiri
Copy link

Raikiri commented Dec 9, 2020

@RoyBerube yes, I had very similar experience. However, I sometimes had more weirdness when an arc welder with UPS mod just kept showing very little charges in crafting menu (like 20 or so), so I could not start a craft with it even though there was a UPS around.

There's even more weirdness to this problem coming from a CBM that gives integrated UPS -- I literally could not make it work in crafting at all, but it can be a separate issue.

@kevingranade kevingranade added this to the 0.F milestone Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling <Exploit> Unintended interactions or behavior, usually breaking how the game is balanced (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants