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

Nerfed the hell out of memory update overhead #38547

Merged
merged 1 commit into from
Mar 4, 2020

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Fixes #37072
Bionic memory bank power usage is super high, at very high skill levels it looks like this can kick in about once a minute or so, and at 25kJ an activation, that drains bionic power absurdly quickly.

Describe the solution

Massively dropped power consumption of memory bank CBM, from 25kJ an activation to 25J an activation. That's roughly 60W of continuous power consumption, which is still pretty high, but we're basically saying it's analagous to a DRAM memory refresh over a stupendous number of memory cells.

Describe alternatives you've considered

IDK, I don't think there's a super reasonable way to balance this CBM as written, so all I'm trying to do here is pound out an utterly insane outcome.

Testing

None, I'm not worried about this having side effects.

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Bionics CBM (Compact Bionic Modules) labels Mar 4, 2020
@ZhilkinSerg ZhilkinSerg merged commit e74e70c into master Mar 4, 2020
@kevingranade kevingranade deleted the kevingranade-inerfedrust branch March 30, 2020 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bionics CBM (Compact Bionic Modules) <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Skill decay and Memory banks
2 participants