Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

feat: add clock (time) #288

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Conversation

Freyskeyd
Copy link
Member

@Freyskeyd Freyskeyd commented Aug 29, 2023

Description

This PR initiates the work on the state sync by introducing a clock in order to follow the block production. For now we use a time based clock.

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch 2 times, most recently from 50bc298 to 042e5d7 Compare August 30, 2023 09:41
@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Patch coverage: 94.80% and project coverage change: +0.23% 🎉

Comparison is base (93d68a5) 52.97% compared to head (a79ec32) 53.21%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #288      +/-   ##
==========================================
+ Coverage   52.97%   53.21%   +0.23%     
==========================================
  Files         214      216       +2     
  Lines       10522    10593      +71     
==========================================
+ Hits         5574     5637      +63     
- Misses       4948     4956       +8     
Files Changed Coverage Δ
crates/topos-p2p/src/constant.rs 66.66% <ø> (ø)
crates/topos-p2p/src/behaviour/gossip.rs 85.32% <66.66%> (-0.53%) ⬇️
crates/topos-clock/src/lib.rs 93.75% <93.75%> (ø)
crates/topos-clock/src/time.rs 96.49% <96.49%> (ø)
crates/topos-p2p/src/network.rs 82.01% <100.00%> (-0.13%) ⬇️

... and 1 file with indirect coverage changes

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

@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch 2 times, most recently from d9fe733 to c48dd9a Compare September 4, 2023 12:57
@Freyskeyd Freyskeyd marked this pull request as ready for review September 4, 2023 12:59
@Freyskeyd Freyskeyd requested a review from a team as a code owner September 4, 2023 12:59
@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch 2 times, most recently from f75da03 to 07de236 Compare September 4, 2023 13:15
crates/topos-clock/src/time.rs Show resolved Hide resolved
crates/topos-clock/Cargo.toml Show resolved Hide resolved
crates/topos-clock/Cargo.toml Outdated Show resolved Hide resolved
crates/topos-clock/src/lib.rs Outdated Show resolved Hide resolved
crates/topos-clock/src/time.rs Show resolved Hide resolved
crates/topos-clock/src/time.rs Outdated Show resolved Hide resolved
crates/topos-clock/src/lib.rs Outdated Show resolved Hide resolved
crates/topos-clock/src/lib.rs Outdated Show resolved Hide resolved
@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch from 07de236 to c2a2c3b Compare September 11, 2023 13:21
crates/topos-clock/src/time.rs Outdated Show resolved Hide resolved
crates/topos-clock/src/time.rs Show resolved Hide resolved
crates/topos-clock/src/lib.rs Show resolved Hide resolved
crates/topos-clock/src/time.rs Show resolved Hide resolved
crates/topos-clock/src/lib.rs Show resolved Hide resolved
@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch 2 times, most recently from 4ec3a44 to 53f688b Compare September 11, 2023 13:53
Signed-off-by: Simon Paitrault <[email protected]>
@Freyskeyd Freyskeyd force-pushed the feature/state-sync-first-iteration branch from 53f688b to a79ec32 Compare September 12, 2023 06:49
@Freyskeyd Freyskeyd merged commit 16ec376 into main Sep 12, 2023
20 checks passed
@Freyskeyd Freyskeyd deleted the feature/state-sync-first-iteration branch September 12, 2023 09:00
gruberb pushed a commit that referenced this pull request Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants