refactor: remove PartialEq from PTCCache

PTCCache comparison excluded via #[derivative(PartialEq = "ignore")]
on BeaconState.ptc_cache field (matching cache field pattern).

Removes unused derivative PartialEq implementation and comparison functions.

Author: Subhasish-Behera

types/src/gloas/ptc_cache.rs

@@ -1,52 +1,23 @@
 use std::{collections::HashMap, sync::Arc};
 
-use derivative::Derivative;
 use once_cell::sync::OnceCell;
 
 use crate::phase0::primitives::{Epoch, Slot, ValidatorIndex};
 
-/// Equivalence function for `ptc_positions` that ignores trailing `None` entries.
-/// Will be used in tests.
-fn compare_ptc_positions(xs: &Vec<Option<usize>>, ys: &Vec<Option<usize>>) -> bool {
-    use std::cmp::Ordering;
-
-    let (shorter, longer) = match xs.len().cmp(&ys.len()) {
-        Ordering::Equal => {
-            return xs == ys;
-        }
-        Ordering::Less => (xs, ys),
-        Ordering::Greater => (ys, xs),
-    };
-
-    shorter == &longer[..shorter.len()]
-        && longer[shorter.len()..].iter().all(|new| new.is_none())
-}
-
-/// Equivalence function for `multiple_positions` HashMap.
-fn compare_multiple_positions(
-    xs: &HashMap<ValidatorIndex, Vec<(Slot, usize)>>,
-    ys: &HashMap<ValidatorIndex, Vec<(Slot, usize)>>,
-) -> bool {
-    xs == ys
-}
-
 /// Cache for Payload Timeliness Committee (PTC) assignments for an entire epoch.
 ///
 /// Uses a hybrid storage approach:
 /// - Most validators have few occurrences: stored in flat Vec
 /// - Validators with multiple occurrences: stored in HashMap
-#[derive(Clone, Debug, Derivative)]
-#[derivative(PartialEq)]
+#[derive(Clone, Debug)]
 pub struct PTCCache {
     /// The epoch this cache was initialized for
     pub initialized_epoch: Option<Epoch>,
     /// Flat shuffling for all slots in epoch
     pub ptc_shuffling: Vec<ValidatorIndex>,
     /// Reverse index for single occurrences: validator_index → first position
-    #[derivative(PartialEq(compare_with = "compare_ptc_positions"))]
     pub ptc_positions: Vec<Option<usize>>,
     /// HashMap for validators with multiple occurrences
-    #[derivative(PartialEq(compare_with = "compare_multiple_positions"))]
     pub multiple_positions: HashMap<ValidatorIndex, Vec<(Slot, usize)>>,
     /// PTC size per slot
     pub ptc_size: usize,