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

Cable charger CBM don't draw power when you're full #37290

Merged
merged 1 commit into from
Jan 24, 2020

Conversation

Fris0uman
Copy link
Contributor

@Fris0uman Fris0uman commented Jan 21, 2020

Summary

SUMMARY: Balance "Cable charger CBM don't draw power when you're full"

Purpose of change

Cable charger CBM don't draw power when you're full.
I'm pretty sure a cable connecting a power source to a full battery will not draw power from the power source.

Describe the solution

If power bank are full don't consume remote fuel

Describe alternatives you've considered

Testing

Compile without error
Get Cable Charger CBM
Plug to UPS
Disable fuel saving mode
Wait
Power banks reach max
Keep waiting
UPS charges don't go down
Repeat with vehicle
Power goes up to max
Keep waiting
No power consumed from vehicle

Additional context

This is on top of fuel saving mode, if you turn the mod off and keep the cable cahrger on you will still not waste power.
Maybe I should remove code to support fuel saving mode on the cable charger in favour of that?

@Fris0uman Fris0uman marked this pull request as ready for review January 22, 2020 18:11
@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Bionics CBM (Compact Bionic Modules) Game: Balance Balancing of (existing) in-game features. labels Jan 24, 2020
@ZhilkinSerg ZhilkinSerg merged commit 50ab9da into CleverRaven:master Jan 24, 2020
@Fris0uman Fris0uman deleted the charger_no_waste branch January 24, 2020 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bionics CBM (Compact Bionic Modules) [C++] Changes (can be) made in C++. Previously named `Code` Game: Balance Balancing of (existing) in-game features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants