-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Pre-generated SRS loading #328
Conversation
src/bin/avail-light.rs
Outdated
@@ -192,7 +192,7 @@ async fn run(error_sender: Sender<anyhow::Error>) -> Result<()> { | |||
#[cfg(feature = "network-analysis")] | |||
tokio::task::spawn(network_analyzer::start_traffic_analyzer(cfg.port, 10)); | |||
|
|||
let pp = Arc::new(kate_recovery::testnet::public_params(1024)); | |||
let pp = Arc::new(kate_recovery::testnet_v2::public_params()); |
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.
Let's use the Couscous?
Cargo.lock
Outdated
"sp-arithmetic 16.0.0", | ||
"sp-core 21.0.0", | ||
"sp-io 23.0.0", | ||
"sp-arithmetic 7.0.0", |
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 think we should revert changes which reduces versions to old ones, it seems that every update switches to old/new versions randomly.
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.
Ah, I see, No idea why this is happening. but will revert it
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.
Also, I think we should update public parameters in test, but not 100% sure.
Companion PRs:
avail-core PR#58
avail PR#270