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

feat!: Allow downstream users to avoid the precomputations that are needed for the FixedBaseMSM algorithm in FK20 #224

Merged
merged 7 commits into from
Aug 15, 2024

Conversation

kevaundray
Copy link
Contributor

@kevaundray kevaundray commented Aug 15, 2024

Breaking change

Users now need to specify whether they want to use Precomp or not, if calling DasContext::with_threads. To preserve the previous behaviour, one can pass UsePrecomp::Yes{width:8}

Its also breaking because the Default::default method no longer creates the precomputations. This was changed because the Default::default method was using up 100MB of data, and we want the user to intentionally opt into this.

Note

In the future, we may be able to allow the user to specify a memory-limit and we compute the width parameter based off of that, but this seems like a non-goal right now

@kevaundray kevaundray changed the title feat!: Allow downstream users to not use precomputations feat!: Allow downstream users to avoid the precomputations that are needed for the FixedBaseMSM algorithm in FK20 Aug 15, 2024
@kevaundray kevaundray added this pull request to the merge queue Aug 15, 2024
Merged via the queue into master with commit 5367e79 Aug 15, 2024
41 checks passed
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.

1 participant