Skip to content
This repository was archived by the owner on Jun 29, 2022. It is now read-only.

hashmap: document Filecoin HAMT variations #282

Closed
wants to merge 2 commits into from
Closed

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Jul 22, 2020

I believe this is correct as of now, but there's some potential modifications that may happen prior to mainnet.

This section will get linked from https://github.com/ipfs/go-hamt-ipld/pull/52 and the Filecoin specs which currently just point to the top of this doc.

Copy link
Member

@vmx vmx left a comment

Choose a reason for hiding this comment

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

Thanks for documenting the Filecoin variation. I haven't checked if the schema matches the Rust implementation as it would be more time consuming than I anticipated.

@rvagg
Copy link
Member Author

rvagg commented Jul 22, 2020

re serialization in Rust:

👍

@rvagg
Copy link
Member Author

rvagg commented Jul 27, 2020

They've switched to SHA2-256 for hashAlg upstream so the whole murmur3 hassle has gone away. Have updated the PR to match.

@rvagg
Copy link
Member Author

rvagg commented Jul 27, 2020

932af7c

@rvagg rvagg closed this Jul 27, 2020
@rvagg rvagg deleted the rvagg/filecoin-hamt branch July 27, 2020 21:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants