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

Update OpenAPI schemas according to OPTIMADE Python tools v0.13.3 #352

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

CasperWA
Copy link
Member

@CasperWA CasperWA commented Mar 4, 2021

This is yet another update to the OpenAPI schemas.

The changes are related to the update of the Python package used to create the schemas, pydantic, where OPTIMADE Python tools now depends on the latest version (v1.8.1) instead of being fixed at an older version (v1.6).

The related OPTIMADE Python tools PR: Materials-Consortia/optimade-python-tools#731.
The changes are related to enumerations (Python Enum sub-classes), extending information about them and defining the default value for the aggregate field attribute for a links resource.

@CasperWA CasperWA added PR/minor-fixes-only PR/ready-for-review Add this flag if you are the author of the PR and you want it to be reviewed. Remove it when editing labels Mar 4, 2021
@ml-evs
Copy link
Member

ml-evs commented Mar 10, 2021

I'm holding off reviewing this as I'm quite closely involved in the changes...

@CasperWA
Copy link
Member Author

I'm holding off reviewing this as I'm quite closely involved in the changes...

So you don't agree with them? 😏

Copy link
Contributor

@rartino rartino left a comment

Choose a reason for hiding this comment

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

The changes look reasonable to me. It isn't super easy to properly review these, as that would require importing them and run validations against them (which I did not do). I think that eventually I will be able to do that with my implementation, though.

@CasperWA
Copy link
Member Author

(...) I think that eventually I will be able to do that with my implementation, though.

Intriguing. Will this implementation be part of a repo in this GH organization or is it something different?

@CasperWA CasperWA merged commit 37ee13b into Materials-Consortia:develop Mar 16, 2021
@CasperWA CasperWA deleted the update_schemas branch March 16, 2021 13:09
@rartino
Copy link
Contributor

rartino commented Mar 16, 2021

Intriguing. Will this implementation be part of a repo in this GH organization or is it something different?

There will be a validator as part of the httk OPTIMADE implementation capable of validating responses based on a schema (there actually already was such a validator in the old optimade-python-candidate-reference-implementation...)

ml-evs pushed a commit that referenced this pull request Apr 23, 2021
The related OPTIMADE Python tools PR:
Materials-Consortia/optimade-python-tools#731.
The changes are related to enumerations (Python Enum
sub-classes), extending information about them and defining
the default value for the aggregate field attribute for a links
resource.
ml-evs pushed a commit that referenced this pull request May 30, 2021
The related OPTIMADE Python tools PR:
Materials-Consortia/optimade-python-tools#731.
The changes are related to enumerations (Python Enum
sub-classes), extending information about them and defining
the default value for the aggregate field attribute for a links
resource.
rartino pushed a commit that referenced this pull request Jul 7, 2021
The related OPTIMADE Python tools PR:
Materials-Consortia/optimade-python-tools#731.
The changes are related to enumerations (Python Enum
sub-classes), extending information about them and defining
the default value for the aggregate field attribute for a links
resource.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/minor-fixes-only PR/ready-for-review Add this flag if you are the author of the PR and you want it to be reviewed. Remove it when editing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants