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

bake known checkpoints #3227

Open
aatifsyed opened this issue Jul 18, 2023 · 1 comment
Open

bake known checkpoints #3227

aatifsyed opened this issue Jul 18, 2023 · 1 comment
Labels
Priority: 4 - Low Limited impact and can be implemented at any time Ready Issue is ready for work and anyone can freely assign it to themselves Type: Enhancement

Comments

@aatifsyed
Copy link
Contributor

aatifsyed commented Jul 18, 2023

I'd like to remove the serialization for this, and turn it into rust code, probably using something like databake, but that's a little heavyweight - we could then just

struct KnownBlocks<'a> {
    calibnet: &'a [(i64, Cid)],
    mainnet: &'a [(i64, Cid)],
}
const KNOWN_BLOCKS: KnownBlocks<'static> = todo!();

Unfortunately, multihash and cid aren't there yet:
multiformats/rust-cid#138
multiformats/rust-multihash#330

Originally posted by @aatifsyed in #3220 (comment)

@aatifsyed
Copy link
Contributor Author

Those issues are closed as wrong, we can proceed with this, see multiformats/rust-multihash#331 (comment)

@aatifsyed aatifsyed added the Ready Issue is ready for work and anyone can freely assign it to themselves label Aug 1, 2023
@lemmih lemmih added Priority: 4 - Low Limited impact and can be implemented at any time Type: Enhancement labels Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 4 - Low Limited impact and can be implemented at any time Ready Issue is ready for work and anyone can freely assign it to themselves Type: Enhancement
Projects
None yet
Development

No branches or pull requests

2 participants