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

Use 1.2 grammar by default and relax rules on nested fields #2242

Merged
merged 4 commits into from
Mar 16, 2025

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented Mar 8, 2025

This PR simply removes the "develop" tag from the v1.2.0 grammar, which should make it the default.

It also allows nested field names to have more flexibility, inspired by e.g., the MP field _mp_stability.gga_gga+u_r2scan (#2182). The spec reads that dictionary keys must only be strings, not identifiers, so I have relaxed our previous approach of enforcing an identifier to allow for filtering on such fields. I'll do a bit of testing to make sure that this is still URL-safe, but I think we should be fine.

@ml-evs
Copy link
Member Author

ml-evs commented Mar 8, 2025

pre-commit.ci autofix

@ml-evs ml-evs force-pushed the ml-evs/v1.2-grammar branch from b8226f9 to 51336b6 Compare March 8, 2025 17:01
@ml-evs
Copy link
Member Author

ml-evs commented Mar 8, 2025

pre-commit.ci autofix

Copy link

codecov bot commented Mar 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.61%. Comparing base (0be0c7f) to head (f1b0731).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2242   +/-   ##
=======================================
  Coverage   90.60%   90.61%           
=======================================
  Files          75       75           
  Lines        4929     4931    +2     
=======================================
+ Hits         4466     4468    +2     
  Misses        463      463           
Flag Coverage Δ
project 90.61% <100.00%> (+<0.01%) ⬆️
validator 90.61% <100.00%> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ml-evs ml-evs force-pushed the ml-evs/v1.2-grammar branch from 4f11c24 to 389353a Compare March 8, 2025 17:21
@ml-evs
Copy link
Member Author

ml-evs commented Mar 8, 2025

pre-commit.ci autofix

@ml-evs ml-evs force-pushed the ml-evs/v1.2-grammar branch from 325d059 to f35da27 Compare March 9, 2025 13:37
@ml-evs ml-evs marked this pull request as ready for review March 16, 2025 21:54
@ml-evs ml-evs requested a review from CasperWA as a code owner March 16, 2025 21:54
ml-evs added 2 commits March 16, 2025 21:57
Bump number of test properties

Add skip for elasticsearch test
@ml-evs ml-evs force-pushed the ml-evs/v1.2-grammar branch from 000d7a2 to f1b0731 Compare March 16, 2025 21:57
@ml-evs ml-evs merged commit 9736e46 into main Mar 16, 2025
13 checks passed
@ml-evs ml-evs deleted the ml-evs/v1.2-grammar branch March 16, 2025 22:13
@ml-evs ml-evs added grammar Concerns the Lark grammar files OPTIMADE v1.2 This label describes actions that have to be taken to be complient with OPTIMADE 1.2 labels Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grammar Concerns the Lark grammar files OPTIMADE v1.2 This label describes actions that have to be taken to be complient with OPTIMADE 1.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant