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

Remove UPS Conversion Mod #29976

Closed

Conversation

esotericist
Copy link
Contributor

@esotericist esotericist commented Apr 27, 2019

Summary

SUMMARY: Content "Remove UPS Conversion Mod"

Purpose of change

With #29651 , most tools no longer have any inherent battery storage. Consequently, applying a UPS Conversion to a tool causes it to be unable to take any charge (due to no longer accepting a magazine). The item mod is also functionally obsolete, since its primary purpose was to make tools rechargable in a UPS-Compatible Recharging Station, and this is already the case (as all current battery cells are rechargable).

Fixes #29961

Describe the solution

Removed UPS Conversion Mod, its recipe, and all mentions of it in any other files in the tree (including mainlined game mods).
Also added it to migration.json, transforming to a small storage battery (the most 'valuable' component used in crafting the mod).

Describe alternatives you've considered

It's possible to give the UPS conversion mod a degree of power storage, enabling it to serve some purpose in light of the current state of non-vehicle-charger UPS ignoring battery cells, but I'm not sure how the best way to do that is (or if it's even desirable).

@DracoGriffin
Copy link
Contributor

Removing this doesn't seem like an appropriate fix, unless there is some plan to restore it somehow else?

@esotericist
Copy link
Contributor Author

esotericist commented Apr 27, 2019

Removing this doesn't seem like an appropriate fix, unless there is some plan to restore it somehow else?

Battery cells can already be recharged in a UPS-compatible recharger, which was the primary purpose of the conversion mod.

Edit: I meant to ask with that: what is the purpose you see in restoring it?

@HKMist
Copy link

HKMist commented Apr 27, 2019

UPS Conversion Mod is just break by the battery update and not obsoleted.

One of the main usage of ups conversion is to make tools run on standard battery to able to directly power by UPS/ Bionic power with UPS upgrade, the new battery system just annoy this function of the mod.

Item install with ups conversion mod before the battery update now functioning in a strange way, it will drain its own battery first, when its own battery is out it will start draining UPS/Bionic power until it is off, And if the item is turn off, it won't turn on again until it is recharge even with a UPS.

Unload a battery to recharge is good but it can't/shouldn't replace the direct supply function. And outright remove everything relay to the mod seem too early, it is better trying to fix it or at least add something to replace the function of the mod first.

@esotericist
Copy link
Contributor Author

Unload a battery to recharge is good but it can't/shouldn't replace the direct supply function. And outright remove everything relay to the mod seem too early, it is better trying to fix it or at least add something to replace the function of the mod first.

Wouldn't it be preferable to make the UPS devices/CBM directly charge chargable cells? They've already been established as being UPS-compatible by the fact they can be charged in a UPS-compatible recharging station.

@esotericist
Copy link
Contributor Author

Didn't expect the conversion mod to feature in tests. I'll look into that later.

@egorzaderiy
Copy link
Contributor

Isn't the primary function of UPS mod is to allow directly charge tools withour battery reloading? At least this functionality needs to be restored, isn't it?

@HKMist
Copy link

HKMist commented Apr 27, 2019

Yes, let UPS charge battery directly is a solution but honestly I think it will involve some bigger problem later.
First, You will not want to auto recharge everything in your bag, CBM can be off but UPS device can't, so if you have a electric jackhammer(not UPS modded) and a laser rifle, before the update you would have any trouble. But if UPS auto recharge everything run on battery, you will need to do some extra click on power management.
Second, I think there are some people don't like the idea of all battery being rechargeable and going to made a mod to separate battery to rechargeable and not rechargeable later. :P

@esotericist
Copy link
Contributor Author

Second, I think there are some people don't like the idea of all battery being rechargeable and going to made a mod to separate battery to rechargeable and not rechargeable later. :P

For what it's worth, more batteries are almost certainly coming, including non-rechargables. The big chance recently was on getting the game to use batteries as magazines in the first place.

Your other points are pertinent, though.

@esotericist
Copy link
Contributor Author

Closing, to approach this in a different way (as well as fix other broken things).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UPS conversion mod don't work
4 participants