Skip to content

Ensure C implementation of CRC32C package is used #408

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.

Sign up for GitHub

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

Jump to bottom
Open
cortadocodes opened this issue Mar 30, 2022 · 3 comments
Open

Ensure C implementation of CRC32C package is used #408

cortadocodes opened this issue Mar 30, 2022 · 3 comments
Assignees
@cortadocodes
Labels
performance If you want to run cool, you've got to run on heavy, heavy fuel

Comments

@cortadocodes
Copy link
Member

This is much faster than the pure-python implementation. Currently, SLOW_CRC32C_WARNING are raised in the data-gateway package and possibly elsewhere.
@cortadocodes cortadocodes added the performance If you want to run cool, you've got to run on heavy, heavy fuel label Mar 30, 2022
@cortadocodes
Copy link
Member Author

I think this was just an issue on my own development machine as the optimised wheel may not be available for it yet.

@cortadocodes cortadocodes moved this to Done in Octue Board May 3, 2022
@cortadocodes cortadocodes reopened this Jan 13, 2025
@cortadocodes
Copy link
Member Author

This shows up every time the octue CLI is used

@cortadocodes cortadocodes moved this from Done to Priority 1 (Low) in Octue Board Jan 13, 2025
@cortadocodes cortadocodes self-assigned this Jan 13, 2025
@cortadocodes
Copy link
Member Author

Update: I've suppressed the warnings for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cortadocodes cortadocodes

Labels
performance If you want to run cool, you've got to run on heavy, heavy fuel
Projects
Octue Board
Status: Priority 1 (Low)
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thclark @cortadocodes