add todo

succinctlabs · Sep 17, 2024 · cfb3d4e · cfb3d4e
1 parent 45384d6
commit cfb3d4e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/scripts/prove/bin/cost_estimator.rs b/scripts/prove/bin/cost_estimator.rs
@@ -117,7 +117,8 @@ struct BatchHostCli {
 }
 
 fn get_max_span_batch_range_size(chain_id: u64) -> u64 {
-    const DEFAULT_SIZE: u64 = 50;
+    // TODO: The default size/batch size should be dynamic based on the L2 chain. Specifically, look at the gas used across the block range (should be fast to compute) and then set the batch size accordingly.
+    const DEFAULT_SIZE: u64 = 1000;
     match chain_id {
         8453 => 5,      // Base
         11155111 => 20, // OP Sepolia

diff --git a/utils/host/src/fetcher.rs b/utils/host/src/fetcher.rs
@@ -26,6 +26,7 @@ use crate::{
 #[derive(Clone)]
 /// The OPSuccinctDataFetcher struct is used to fetch the L2 output data and L2 claim data for a
 /// given block number. It is used to generate the boot info for the native host program.
+/// TODO: Add retries for all requests (3 retries).
 pub struct OPSuccinctDataFetcher {
     pub l1_rpc: String,
     pub l1_provider: Arc<RootProvider<Http<Client>>>,