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

mv reward calculations from end block #457

Conversation

akremstudy
Copy link
Contributor

No description provided.

@akremstudy akremstudy marked this pull request as ready for review December 9, 2024 13:53
@akremstudy akremstudy merged commit e066c09 into explore-a-running-aggregate Dec 9, 2024
7 checks passed
@akremstudy akremstudy deleted the remove-reward-calculation-from-endblock branch December 9, 2024 13:53
brendaloya pushed a commit that referenced this pull request Dec 11, 2024
* chore: remove unused files

* reduce for loops in WeightedMode

* explore removing aggregation from endBlock

* add different indexes, fix tests plus more

* uncomment test

* add case to TestAddReport

* add test, rename fields, cleanup

* remove aggregate_report_index field

* chore: cleanup comments and prints

* fix cyclelist rewards

* block bridge deposit report for claimed ids

* rate limit snapshot creation

* wip test

* chore: remove unused files

* mv reward calculations from end block (#457)

* first commit

* refactor and fix getter

* handle leftover fraction plus tests

* change docker image for test

* fix workflow cfg

* separate workflow jobs

* fix x/reporter/autocli

* lint

* upgrade handling

* register handler

* lint

* add msg_server_withdraw_tip_legacy for tips before
upgrade

* fix migrations method

* lint

* fix migrations

* add query to fetch report ids and query_ids

* small test

* add query to autocli
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.

2 participants