Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Slight refactor #7

Closed
wants to merge 13 commits into from
Closed

Slight refactor #7

wants to merge 13 commits into from

Conversation

feliam
Copy link
Collaborator

@feliam feliam commented Jan 8, 2025

  • Metric is a separated entity that gets updated similar to how metrics are updated
  • checks are calculated based on pre/post states:
    pub async fn run(&self, pre_state: &State, post_state: &State) -
  • metrics are updated the same way:
    pub fn update(&self, pre_state: &State, post_state: &State)
  • A single current_state is maintained in the mainloop
    let current_state = Arc::new(RwLock::new(state));
  • Initial height from config or tip
  • States attempt to be at the blockchain latest height (Expects blocks are consecutive (can not skip blocks))
  • lru of blocks disabled.
  • Added metric that checks max estimated block time

@feliam feliam marked this pull request as ready for review January 13, 2025 18:46
@feliam feliam requested a review from Fraccaman January 13, 2025 18:51
@feliam feliam closed this Jan 22, 2025
@feliam feliam deleted the felipe/refactor branch February 18, 2025 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant