Count deals from sealed sectors only

filecoin-project · Dec 3, 2020 · 610e783 · 610e783
1 parent 7a28712
commit 610e783
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/cmd/lotus-shed/sr2-dealstats-rollup.go b/cmd/lotus-shed/sr2-dealstats-rollup.go
@@ -245,11 +245,13 @@ var rollupDealStatsCmd = &cli.Command{
 
 		for dealID, dealInfo := range deals {
 
-			// Counting no-longer-active deals as per Pooja's request
-			// // https://github.com/filecoin-project/specs-actors/blob/v0.9.9/actors/builtin/market/deal.go#L81-L85
-			// if d.State.SectorStartEpoch < 0 {
-			// 	continue
-			// }
+			// Only count deals that have properly started, not past/future ones
+			// https://github.com/filecoin-project/specs-actors/blob/v0.9.9/actors/builtin/market/deal.go#L81-L85
+			// Bail on 0 as well in case SectorStartEpoch is uninitialized due to some bug
+			if dealInfo.State.SectorStartEpoch <= 0 ||
+				dealInfo.State.SectorStartEpoch > head.Height() {
+				continue
+			}
 
 			clientAddr, found := resolvedWallets[dealInfo.Proposal.Client]
 			if !found {