feat(consensus): redefine consensus height metric to working height

[matan-starkware]

This will be more useful than stating the most recent decision when we are stuck on block 0.
Also moved out of Context::decision due to sync

---

This change is 

[matan-starkware]

• feat(consensus): add metric for sync in consensus and update the simulation script #490
• refactor(consensus): rename notify_decision to decision_reached #465
• feat(consensus): simulations with the test config will use the fake consensus sync #414
• feat(consensus): add sync sender to context to broadcast on decision #413
• feat(consensus): add sync_topic to test config #412
• refactor(consensus): consensus handles its network registrations on its own #409
• refactor(consensus): rename topic in config to network_topic #406
• feat(consensus): redefine consensus height metric to working height #404 👈
• feat(consensus): add sync receiver to consensus #389 : 1 other dependent PR (#403 )
• feat(consensus): add decision fn to consensus context API #387
• refactor(consensus): move run_consensus out of lib.rs #386
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on Graphite

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.63%. Comparing base (8dda046) to head (5bf4454).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   76.63%   76.63%           
=======================================
  Files         346      346           
  Lines       36280    36279    -1     
  Branches    36280    36279    -1     
=======================================
  Hits        27803    27803           
+ Misses       6174     6173    -1     
  Partials     2303     2303           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[asmaastarkware]

you should make changes in run_consensus.py, in case one node got stuck and didn't start (now height = 0 not None)

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)

[asmaastarkware]

Reviewed 1 of 1 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)

[asmaastarkware]

Reviewed 2 of 2 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)

[matan-starkware]

Merge activity

• Aug 15, 3:00 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.
• Aug 15, 3:01 AM EDT: @matan-starkware merged this pull request with Graphite.