Skip to content

Conversation

@LadyChristina
Copy link
Member

@LadyChristina LadyChristina commented Mar 16, 2025

All Submissions:

  • Have you followed the guidelines in our Contributing documentation?
  • Have you verified that there aren't any other open Pull Requests for the same update/change?
  • Does the Pull Request pass all tests?

Description

Made various changes - most importantly:

  • Added a configurable parameter that specifies how many windows to look forwards and backwards when determining the population of active block producers in a given time period ('population_windows' in config.yaml)
  • Made Ethereum parser more efficient by using generators instead of lists (because Ethereum block data was getting too large to handle that way)
  • Updated the directory structure of the results and intermediate files
  • Generalised max power ratio metric to concentration ratios
  • Updated documentation

Copy link
Contributor

@ZeeshanJan ZeeshanJan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LadyChristina LadyChristina merged commit e60e87a into main May 6, 2025
1 check passed
@LadyChristina LadyChristina deleted the various_updates branch May 6, 2025 10:54
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.

3 participants