Skip to content

Battery Buffer Tweaks#302

Closed
TechLord22 wants to merge 3 commits intomasterfrom
buffer_tweaks
Closed

Battery Buffer Tweaks#302
TechLord22 wants to merge 3 commits intomasterfrom
buffer_tweaks

Conversation

@TechLord22
Copy link
Copy Markdown
Member

Changes 9 Slot buffers to 8 slots. Evenly distributes energy over all batteries when draining and filling.

@TechLord22 TechLord22 added the status: high priority Issue or PR should be prioritized for reviews label Dec 6, 2021
@TechLord22 TechLord22 requested a review from serenibyss December 6, 2021 22:51
if (distributed <= 0)
return;

long amperageUsed = energyContainer.acceptEnergyFromNetwork(outFacing.getOpposite(), distributed / getOutputAmperage(), getOutputAmperage());
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be distributed * getOutputAmperage()?
(described issue on discord)

private final BitSet batterySlotsUsedThisTick = new BitSet();
private final int tier;
private long amps = 0;
private long lastEnergyInputPerSec = 0;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you need to update these 4 values for the digital cover

@TechLord22 TechLord22 requested a review from brachy84 December 8, 2021 16:46
@serenibyss serenibyss added status: do not merge PR should not be merged and removed status: high priority Issue or PR should be prioritized for reviews labels Dec 9, 2021
@Syrcan Syrcan mentioned this pull request Dec 10, 2021
@serenibyss
Copy link
Copy Markdown
Member

Superseded by #323

@serenibyss serenibyss closed this Dec 12, 2021
@serenibyss serenibyss deleted the buffer_tweaks branch December 12, 2021 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: do not merge PR should not be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants