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

[DO NOT MERGE] Diff between dev and base branch #62

Draft
wants to merge 147 commits into
base: main
Choose a base branch
from

Conversation

SebastianElvis
Copy link
Member

No description provided.

maurolacy and others added 30 commits June 5, 2024 21:17
* Init
* Update babylon-private reference
* Revert "CI: Remove redundant SSH key logic (#218)"
* Add create and check FPs by chain support
* Add consumer FP registration check
* go lint
* Improve error handling
* Make chainID string
* Adjust comment
When error occurs, `block` is `nil` and `block.Height` will cause a nil
pointer panic. Uncovered in local deployment
lesterli and others added 30 commits August 28, 2024 11:25
# Summary
context in
[#496](babylonchain/finality-provider#496)
This pr implements the logic of querying voting power in Op. 

> instead of querying the exact voting power, the FP queries BTC
delegations page by page, and if there is >=1 active BTC delegation,
then it can vote.

# Test plan
```
make test-e2e-op
```

---------

Co-authored-by: Bourne Zhang <[email protected]>
Co-authored-by: lesterli <[email protected]>
This PR merges all latest changes in `main` to the base branch

NOTE:

- num pub rand cannot be the default value of 70000, which is too large
- parallelise e2e
After this PR, we can tag `euphrates-0.5.0-rc.0` for FP
The previous fix #142 only fixed the error handler for babylon
controller chain other than the cosumer chain. This PR addressed it.
Notable changes:
* remove fast sync completely
* introduce batch process for catching up
* introduce signature submission interval to config
* introduce batch submission size to config
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.

9 participants