Remove commitment

EspressoSystems · Sep 14, 2023 · 40bc007 · 40bc007
1 parent 9ed28f3
commit 40bc007
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 22 deletions.
diff --git a/crates/hotshot/src/demos/sdemo.rs b/crates/hotshot/src/demos/sdemo.rs
@@ -236,13 +236,6 @@ impl BlockPayload for SDemoBlock {
                 .collect(),
         }
     }
-
-    fn commitment(&self) -> VidResult<Commitment<Self>> {
-        // TODO: Get the payload commitment after VID integration.
-        // <https://github.com/EspressoSystems/HotShot/issues/1673>
-        // <https://github.com/EspressoSystems/jellyfish/issues/369>
-        unimplemented!();
-    }
 }
 
 impl State for SDemoState {

diff --git a/crates/types/src/traits/block_contents.rs b/crates/types/src/traits/block_contents.rs
@@ -58,14 +58,6 @@ pub trait BlockPayload:
     /// returns hashes of all the transactions in this block
     /// TODO make this ordered with a vec
     fn contained_transactions(&self) -> HashSet<Commitment<Self::Transaction>>;
-
-    /// Compute the VID payload commitment.
-    ///
-    /// # Errors
-    /// - `VidResult::Err` in case of actual error.
-    /// - `VidResult::Ok(Result::Err)` if verification fails.
-    /// - `VidResult::Ok(Result::Ok)` if verification succeeds.
-    fn commitment(&self) -> VidResult<Commitment<Self>>;
 }
 
 /// Abstraction over any type of transaction. Used by [`BlockPayload`].
@@ -159,13 +151,6 @@ pub mod dummy {
         fn contained_transactions(&self) -> HashSet<Commitment<Self::Transaction>> {
             HashSet::new()
         }
-
-        fn commitment(&self) -> VidResult<Commitment<Self>> {
-            // TODO: Get the payload commitment after VID integration.
-            // <https://github.com/EspressoSystems/HotShot/issues/1673>
-            // <https://github.com/EspressoSystems/jellyfish/issues/369>
-            unimplemented!();
-        }
     }
 
     impl TestableBlock for DummyBlock {