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
perf: enable reconstruct trace/commit by default instead of using disk + lower memory usage #323
Conversation
core/src/stark/types.rs
Outdated
@@ -64,7 +64,7 @@ impl<SC: StarkGenericConfig> ShardMainData<SC> { | |||
where | |||
ShardMainData<SC>: Serialize, | |||
{ | |||
let mut writer = BufWriter::new(&file); | |||
let mut writer = BufWriter::with_capacity(1024 * 1024 * 512, &file); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this capacity be some env variable as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going to remove this as I didn't notice any speedup from it.
@@ -476,6 +476,12 @@ pub(super) mod baby_bear_blake3 { | |||
} | |||
} | |||
|
|||
impl Clone for BabyBearBlake3 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where do we need to clone the config?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now it's required in prove_with_config, but maybe it's not necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just removed the bound and it seems fine.
…k + lower memory usage (#323) Co-authored-by: John Guibas <jtguibas@Johns-MBP.monkeybrains.net>
No description provided.