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 meta field to entry model #1698

Conversation

JPBergsma
Copy link
Contributor

@JPBergsma JPBergsma commented Jul 6, 2023

I am trying to add support for the metadata field as described in issue #1697
It is still a work in progress, as I still need to add tests and validators, but comments are welcome.

@codecov
Copy link

codecov bot commented Jul 6, 2023

Codecov Report

Merging #1698 (44d8789) into master (8510ffa) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1698      +/-   ##
==========================================
+ Coverage   90.68%   90.78%   +0.09%     
==========================================
  Files          74       74              
  Lines        4616     4665      +49     
==========================================
+ Hits         4186     4235      +49     
  Misses        430      430              
Flag Coverage Δ
project 90.78% <100.00%> (+0.09%) ⬆️
validator 90.67% <100.00%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
optimade/models/entries.py 98.61% <100.00%> (+0.83%) ⬆️
optimade/server/config.py 94.05% <100.00%> (+0.37%) ⬆️
optimade/server/mappers/entries.py 96.82% <100.00%> (+0.36%) ⬆️
optimade/server/routers/utils.py 95.23% <100.00%> (+0.11%) ⬆️

@ml-evs ml-evs changed the base branch from master to 0.25.x October 23, 2023 12:57
JPBergsma and others added 3 commits November 6, 2023 14:58
* Use Python 3.9 as the 'base' CI version for linting

* Update pre-commit hooks

* Run `pyupgrade --py39-plus` to upgrade legacy annotations

* Add `--exit-non-zero-on-fix` for ruff

Co-authored-by: Casper Welzel Andersen <[email protected]>

* Use f-string over format

Co-authored-by: Casper Welzel Andersen <[email protected]>

---------

Co-authored-by: Casper Welzel Andersen <[email protected]>
@ml-evs ml-evs added the OPTIMADE v1.2 This label describes actions that have to be taken to be complient with OPTIMADE 1.2 label Jun 11, 2024
@ml-evs ml-evs changed the base branch from 0.25.x to ml-evs/add-meta-to-entries October 20, 2024 12:14
@ml-evs ml-evs added the skip_changelog Use this label to omit this PR/issue from the CHANGELOG. label Oct 20, 2024
@ml-evs
Copy link
Member

ml-evs commented Oct 20, 2024

I'm going to merge this into a new branch so that we can work on it again properly

@ml-evs ml-evs merged commit 6eac0a8 into Materials-Consortia:ml-evs/add-meta-to-entries Oct 20, 2024
1 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OPTIMADE v1.2 This label describes actions that have to be taken to be complient with OPTIMADE 1.2 skip_changelog Use this label to omit this PR/issue from the CHANGELOG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants