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

bricks/_common/mpconfigport.h: enable MICROPY_GC_ALLOC_THRESHOLD #267

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

dlech
Copy link
Member

@dlech dlech commented Oct 1, 2024

This can be a useful performance tweak for some applications.

Probably not worth turning on on hubs with smaller RAM though.

@coveralls
Copy link

coveralls commented Oct 1, 2024

Coverage Status

coverage: 56.36%. remained the same
when pulling 40cc061 on dlech:gc-threshold
into 8137ff5 on pybricks:master.

@laurensvalk
Copy link
Member

Probably not worth turning on on hubs with smaller RAM though.

Is it useful on Technic Hub, or perhaps Prime/Inventor/Essential only?

On hubs with PYBRICKS_OPT_EXTRA_LEVEL2.

This can be a useful performance tweak for some applications.
@laurensvalk
Copy link
Member

Merged and enabled for hubs with PYBRICKS_OPT_EXTRA_LEVEL2.

@laurensvalk laurensvalk merged commit 40cc061 into pybricks:master Dec 5, 2024
19 checks passed
@dlech dlech deleted the gc-threshold branch December 5, 2024 15:00
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.

3 participants