Open
Conversation
…tor and bindings guidance
Jake-Moss
reviewed
Dec 18, 2025
|
|
||
| ### 2.4 Matrix datasets: `/data/*` | ||
|
|
||
| Each dataset under `/data` represents one matrix “core”: |
There was a problem hiding this comment.
Do we want to move away from the "matrix core" terminology? It isn't mentioned any where else in this document nor elsewhere on any of the OMX repositories.
As an anecdote, within AequilibraE we've seen some users confuse this term with compute cores/threads, additionally the idea of a "matrix" having multiple "cores" was novel. Instead they used more typical terms, "matrix" and "matrices", to refer to the 2D structures.
My understanding is the "core" term comes from some other piece of software that I am not familiar with.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR introduces a new, normative, versioned core specification for the OMX file format (“OMX 0.2.1") and provides a unified structure for defining:
The goal is to provide a clearer, more rigorous, and more maintainable specification than the current mix of wiki pages ("Data Structure 0.2", "Open matrix requirements", and "Specification v0.2"), which combine design goals, project requirements, and partial format rules.
Key changes:
omx.md- contains the normative language for the OMX specification in the core repo in a way that is declarative and can be version-controlled.readme.mdto refer toomx.mdWhat it does not do:
Things for @bstabler and @billyc to review - in order (if we aren't in agreement about 1, no need to discuss 2 yet!):
Fixes issue #40