Skip to content
This repository was archived by the owner on Jan 8, 2026. It is now read-only.

Conversation

@mikeal
Copy link
Contributor

@mikeal mikeal commented Sep 10, 2019

After commenting on #184 (comment) I realized that we’re missing some of the key motivations for the data model in the spec itself.

This is some draft text on the “block format agnostic” motivation and how it impacts the data model spec.

@mikeal mikeal requested a review from warpfork September 10, 2019 17:51
by common programming languages.** This provides the foundation for block format agnostic tools
to be built using familiar native types in a programmer's preferred language. As such, there
is an element of "lowest common denominator" to the IPLD Data Model in that it cannot support
some advanced features (like non-string keys for Maps) because support for such a feature
Copy link
Member

Choose a reason for hiding this comment

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

could also add "or unsigned integers" inside those parens to drive home the point

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i’d rather not litigate that particular point in this PR :)

@mikeal
Copy link
Contributor Author

mikeal commented Sep 16, 2019

I’m gonna merge in 48 hours unless there are objections.

Copy link
Contributor

@warpfork warpfork left a comment

Choose a reason for hiding this comment

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

Love it.

Stebalien pushed a commit to Stebalien/specs that referenced this pull request Sep 18, 2019
@warpfork warpfork merged commit 8ca5e2c into master Oct 1, 2019
@warpfork warpfork deleted the data-model-motivation branch October 1, 2019 11:26
prataprc pushed a commit to iprs-dev/ipld-specs that referenced this pull request Oct 13, 2020
doc: add motivation section to data model spec
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.

5 participants