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

Unsemantic datamodel #175

Open
jesper-friis opened this issue Oct 26, 2023 · 1 comment
Open

Unsemantic datamodel #175

jesper-friis opened this issue Oct 26, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@jesper-friis
Copy link
Contributor

jesper-friis commented Oct 26, 2023

The species_at_sites property of OPTIMADEStructureAttribute is currently a list of strings, referring to the name of the individual species.

Having species_at_sites being a ref-type to OPTIMADEStructureSpecies would be more semantic and less entangled. This would also be a more efficient way to store the data.

@jesper-friis jesper-friis added the enhancement New feature or request label Oct 26, 2023
@CasperWA
Copy link
Collaborator

That's a fair observation. Again, these data models are taken as verbatim as possible from the OPTIMADE specification, with the limits that this entails as well, of course. See the section on species_at_sites.

I'm sure there are multiple improvements to be made in the data models here.
For example, one could also compact the attributes, combining them with the top-level id and type, eliminating the need for an attributes reference and the OPTIMADEStructureAttributes model (it would effectively be combined with the OPTIMADEStructure model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants