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

Add varint encoding of code to table #297

Open
whyrusleeping opened this issue Oct 25, 2022 · 1 comment
Open

Add varint encoding of code to table #297

whyrusleeping opened this issue Oct 25, 2022 · 1 comment

Comments

@whyrusleeping
Copy link
Member

For a large number of usecases people need to varint encode the multicodec value before making use of it, it might be useful to add that alongside the code in another column.
The failure mode i've observed recently is someone taking the hex for the multicodec, writing it down as bytes, and being confused when things don't line up.

@rvagg
Copy link
Member

rvagg commented Oct 26, 2022

Yeah, fair enough, we had an instance of this kind of confusion recently in a discussion.

The table format is pretty difficult to change at the moment because it's consumed by many automated tools that assume specific columns, and adding a column at the end may not be as useful since it'd be hidden but we've been talking about making a JSON form as the authoritative form which compiles to the CSV, that might be a good place to put it.

@rvagg rvagg moved this to 🏃‍♀️ In Progress in IPLD team's weekly tracker Dec 6, 2022
@rvagg rvagg moved this from 🏃‍♀️ In Progress to 🥞 Todo in IPLD team's weekly tracker Jan 4, 2023
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

No branches or pull requests

3 participants
@rvagg @whyrusleeping and others