Skip to content

Commit

Permalink
temp: remove some validations
Browse files Browse the repository at this point in the history
  • Loading branch information
rvagg committed Jun 3, 2024
1 parent 308f0c0 commit 8208b8e
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions actors/miner/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1961,13 +1961,6 @@ impl Actor {

if params.sectors.is_empty() {
return Err(actor_error!(illegal_argument, "batch empty"));
} else if params.sectors.len() > policy.prove_commit_ni_sector_batch_max_size {
return Err(actor_error!(
illegal_argument,
"batch of {} too large, max {}",
params.sectors.len(),
policy.pre_commit_sector_batch_max_size
));
}

if !can_prove_commit_ni_seal_proof(rt.policy(), params.seal_proof_type) {
Expand Down Expand Up @@ -2001,14 +1994,16 @@ impl Actor {
params.sector_proofs.len()
));
}
validate_seal_proofs(params.seal_proof_type, &params.sector_proofs)?;
// TODO: skip this for now since the sizes are wrong and it will always fail
// validate_seal_proofs(params.seal_proof_type, &params.sector_proofs)?;
} else {
if params.aggregate_proof_type != Some(RegisteredAggregateProof::SnarkPackV2) {
return Err(actor_error!(
illegal_argument,
"aggregate proof type must be SnarkPackV2"
));
}
// TODO: sizes will be wrong for this and will fail when an aggregate is submitted
validate_seal_aggregate_proof(
&params.aggregate_proof,
params.sectors.len() as u64,
Expand Down

0 comments on commit 8208b8e

Please sign in to comment.