Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support aiida 2.0 #118

Merged

Conversation

qiaojunfeng
Copy link
Collaborator

fix #117

Also apply syntax changes suggested by prospector.
An error for plotting Wannier functions in cube format.
Some times wannier90 does not write `aiida.werr` but
`aiida.node_*.werr`.
Check the wout file is complete, otherwise some times the parser
finish successfully but actually the calculation is unfinished.
The `bands_kpoints` accepts a `KpointsData` containing an explicit list
of kpoints and high-symmetry point labels. It will be converted to
`explicit_kpath` and `explicit_kpath_labels` blocks in the wannier90 win
input file.
Make sure they have the same number of digits for floats.
This allows returing more accurate exit code than a too general werr
exists exit code.
Add an `validate_inputs_base` to allow for more granular control in
workflows using `Wannier90Calculation`.
This can significantly save inodes. However, if one wants to restart
wannier_plot from a RemoteData containing chk file, the restart might
fail since UNK files might not exist. In such case, one can use
inputs.settings.additional_remote_symlink_list to manually specify the
symlinks for UNK files from pw2wannier90 RemoteData.
Need to call `super.on_terminated()`
@qiaojunfeng qiaojunfeng self-assigned this Jul 3, 2023
@qiaojunfeng qiaojunfeng force-pushed the fix/compatibility-aiida-2.0 branch 2 times, most recently from 481bdbb to 3388c73 Compare July 3, 2023 17:50
@qiaojunfeng qiaojunfeng force-pushed the fix/compatibility-aiida-2.0 branch from 3388c73 to 37b64d7 Compare July 3, 2023 17:56
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Patch coverage: 40.24% and project coverage change: -19.08 ⚠️

Comparison is base (b9b0e38) 76.01% compared to head (37b64d7) 56.94%.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #118       +/-   ##
============================================
- Coverage    76.01%   56.94%   -19.08%     
============================================
  Files            9       13        +4     
  Lines          834     1310      +476     
============================================
+ Hits           634      746      +112     
- Misses         200      564      +364     
Impacted Files Coverage Δ
src/aiida_wannier90/workflows/minimal.py 0.00% <0.00%> (ø)
src/aiida_wannier90/parsers/postw90.py 7.78% <7.78%> (ø)
src/aiida_wannier90/calculations/postw90.py 18.12% <18.12%> (ø)
src/aiida_wannier90/utils.py 44.44% <38.88%> (ø)
src/aiida_wannier90/orbitals.py 87.27% <77.77%> (ø)
src/aiida_wannier90/calculations/wannier90.py 90.00% <83.33%> (ø)
src/aiida_wannier90/io/_group_list.py 85.71% <83.33%> (ø)
src/aiida_wannier90/parsers/wannier90.py 79.72% <84.93%> (ø)
src/aiida_wannier90/io/_write_win.py 91.07% <91.54%> (ø)
src/aiida_wannier90/__init__.py 100.00% <100.00%> (ø)
... and 3 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@qiaojunfeng qiaojunfeng merged commit 3aed159 into aiidateam:develop Jul 3, 2023
@qiaojunfeng qiaojunfeng deleted the fix/compatibility-aiida-2.0 branch July 4, 2023 15:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update to aiida-core v2
1 participant