Skip to content

Commit

Permalink
fix: ADVZ Share serde derivation (#405)
Browse files Browse the repository at this point in the history
* fix downstream build break with serde

* update flake.lock

* exclude timer tests from CI
  • Loading branch information
ggutoski authored Nov 10, 2023
1 parent 8c66b70 commit 0c22ae3
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 11 deletions.
18 changes: 9 additions & 9 deletions flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 5 additions & 2 deletions primitives/src/vid/advz.rs
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@ where

/// The [`VidScheme::Share`] type for [`Advz`].
#[derive(Derivative, Deserialize, Serialize)]
#[serde(bound = "Output<H>: Serialize + for<'a> Deserialize<'a>")]
#[derivative(
Clone(bound = ""),
Debug(bound = ""),
Expand Down Expand Up @@ -638,7 +639,8 @@ mod tests {
};
use sha2::Sha256;

#[test]
// #[test]
#[allow(dead_code)]
fn disperse_timer() {
// run with 'print-trace' feature to see timer output
let (payload_chunk_size, num_storage_nodes) = (256, 512);
Expand All @@ -651,7 +653,8 @@ mod tests {
let _ = advz.disperse(&payload_random);
}

#[test]
// #[test]
#[allow(dead_code)]
fn commit_only_timer() {
// run with 'print-trace' feature to see timer output
let (payload_chunk_size, num_storage_nodes) = (256, 512);
Expand Down

0 comments on commit 0c22ae3

Please sign in to comment.