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

Add CIRCUITPY_MESSAGE_COMPRESSION_LEVEL #8492

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

jepler
Copy link
Member

@jepler jepler commented Oct 20, 2023

to trade compile speed & flash size

Initially enable the faster mode on rp2040 and espressif, where there's usually plenty of flash available (these advanced techniques save hundreds to thousands of bytes, which is important on a lot of old samd21 boards but is a drop in the lake of a 4MB flash chip)

Levels are:

  • 1: fastest
  • 3: use qstrs
  • 5: use custom dictionary

Tested on the unix build (level 1) and still passes tests.

real time duration of running just maketranslationdata (not make) on a i5-1335U (newish laptop) for feather_rp2040 for various levels:

  • 1: 0.09s
  • 4: 0.29s
  • 9: 20.33s

to trade compile speed & flash size

Initially enable the faster mode on rp2040 and espressif, where there's
usually plenty of flash available (these advanced techniques save hundreds
to thousands of bytes, which is important on a lot of old samd21 boards
but is a drop in the lake of a 4MB flash chip)
@jepler jepler requested a review from tannewt October 23, 2023 17:33
@tannewt
Copy link
Member

tannewt commented Oct 23, 2023

@eightycc What do you think? How does this interact with your changes?

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Looks good to me! I suspect eightycc will integrate their changes to speed up level 5.

@tannewt
Copy link
Member

tannewt commented Oct 23, 2023

Very curious to see how much faster builds are since this step is run for every board and translation combination.

@tannewt tannewt merged commit f9e63d9 into adafruit:main Oct 23, 2023
@eightycc
Copy link

Looks good to me, too. My changes will fit at level 5.

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