Skip to content

Fixing excessive memory consumption #5

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

Merged
merged 4 commits into from
Nov 20, 2023
Merged

Fixing excessive memory consumption #5

merged 4 commits into from
Nov 20, 2023

Conversation

NWItaly
Copy link
Contributor

@NWItaly NWItaly commented Nov 18, 2023

I use an Arduino Uno R3 and I would like to use CalibrationHelper example, but it didn't compile for excessive memory consuption.

BasicReading

Before: Global variables use 573 bytes (27%) of dynamic memory, leaving 1475 bytes for local variables. Maximum is 2048 bytes.
After: Global variables use 275 bytes (13%) of dynamic memory, leaving 1773 bytes for local variables. Maximum is 2048 bytes.

ManualCalibration

Before: Global variables use 697 bytes (34%) of dynamic memory, leaving 1351 bytes for local variables. Maximum is 2048 bytes.
After: Global variables use 275 bytes (13%) of dynamic memory, leaving 1773 bytes for local variables. Maximum is 2048 bytes.

CalibrationHelper

Before: Global variables use 3023 bytes (147%) of dynamic memory, leaving -975 bytes for local variables. Maximum is 2048 bytes.
After: Global variables use 323 bytes (15%) of dynamic memory, leaving 1725 bytes for local variables. Maximum is 2048 bytes.

@NWItaly NWItaly closed this Nov 18, 2023
@NWItaly NWItaly reopened this Nov 18, 2023
@NWItaly
Copy link
Contributor Author

NWItaly commented Nov 18, 2023

There's not Release_Candidate branch... Link

@sfe-SparkFro sfe-SparkFro marked this pull request as ready for review November 20, 2023 18:58
@sfe-SparkFro
Copy link
Collaborator

Thank you for this, I pay my respects to you!

No worries about the missing branch, we can just merge this into main.

@sfe-SparkFro sfe-SparkFro merged commit d37f144 into sparkfun:main Nov 20, 2023
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.

2 participants