Skip to content

Bugfixes: Blessed entries doc + query#1376

Merged
tsmathis merged 3 commits intodevelopfrom
bugfix/blessed
Jan 9, 2026
Merged

Bugfixes: Blessed entries doc + query#1376
tsmathis merged 3 commits intodevelopfrom
bugfix/blessed

Conversation

@esoteric-ephemera
Copy link
Copy Markdown
Collaborator

  • Fix BlessedCalcs document model to take in RunType.HSE06 data via alias - all of this data is currently None in the DB because of improper aliasing
  • Fix associated query operators:
    • Ensure correct mapping of str or RunType to BlessedCalcs model field
    • Ensure post-processing returns data in the emmet.core.vasp.material.MaterialsDoc schema so that data (the blessed entry itself) is not lost upon deserialization
    • Tests

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 8, 2026

Codecov Report

❌ Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.01%. Comparing base (a03eea6) to head (f27a35e).

Files with missing lines Patch % Lines
.../api/routes/materials/materials/query_operators.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1376      +/-   ##
===========================================
+ Coverage    85.96%   86.01%   +0.04%     
===========================================
  Files          229      229              
  Lines        17888    17898      +10     
===========================================
+ Hits         15378    15395      +17     
+ Misses        2510     2503       -7     

☔ 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.

@tsmathis
Copy link
Copy Markdown
Collaborator

tsmathis commented Jan 8, 2026

Good catch, I count 10k HSE06 run_type tasks with a production build tag, so at least a non-zero number of materials should include HSE blessed entries.

Will be good to get this in the next build

@esoteric-ephemera
Copy link
Copy Markdown
Collaborator Author

Also noting that this is in support of an old client PR for retrieving blessed entries

@tsmathis
Copy link
Copy Markdown
Collaborator

tsmathis commented Jan 9, 2026

This should be good to go, right? Anything else need to be included here?

@esoteric-ephemera
Copy link
Copy Markdown
Collaborator Author

GTG from my side!

@tsmathis tsmathis merged commit 29e9e77 into develop Jan 9, 2026
12 checks passed
@tsmathis tsmathis deleted the bugfix/blessed branch January 9, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants