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

Bugfix/espresso nscf kspacing #2330

Conversation

nerdwarrior
Copy link
Contributor

Summary of Changes

Added kspacing to espresso non_scf_job() calc_defaults to prevent crashes. Previously, nscf jobs (with no other k-point specification via **calc_kwargs) would fail to start without this addition.

N.B. An extra unit test failure is recorded by pytest tests/core/recipes/espresso_recipes/test_core.py with assert "kspacing" not in results["parameters"]. @Andrew-S-Rosen not sure if this should exist in the test in the first place, given that jobs crash without it.

Requirements

Notes

@buildbot-princeton
Copy link
Collaborator

Can one of the admins verify this patch?

@Andrew-S-Rosen
Copy link
Member

@nerdwarrior: Thank you very much for this PR! I agree that this is a logical change, and choosing kspacing: 0.033 seems consistent with the other recipes. Please feel free to update the test to update the kspacing check in that test. We should change it to check that the value is 0.033 per your update.

@Andrew-S-Rosen
Copy link
Member

Thank you! Setting this to auto-merge now.

@Andrew-S-Rosen Andrew-S-Rosen enabled auto-merge (squash) July 8, 2024 17:53
Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.86%. Comparing base (e757741) to head (2598658).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2330   +/-   ##
=======================================
  Coverage   98.86%   98.86%           
=======================================
  Files          85       85           
  Lines        3533     3533           
=======================================
  Hits         3493     3493           
  Misses         40       40           

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

@Andrew-S-Rosen Andrew-S-Rosen merged commit 4c839f4 into Quantum-Accelerators:main Jul 8, 2024
20 checks passed
@nerdwarrior nerdwarrior deleted the bugfix/espresso-nscf-kspacing branch July 8, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants