aiidateam / aiida-pseudo

MIT License
5 stars 8 forks source link

CLI: Add support for units to `family show` #96

Closed mbercx closed 3 years ago

mbercx commented 3 years ago

We've added support for units in https://github.com/aiidateam/aiida-pseudo/pull/57, but forgot to update the family show command. This means that the user still sees the hardcoded default (eV), even when the unit of the configured recommended cutoffs is different:

$ aiida-pseudo family show SSSP/1.1/PBE/efficiency 
Element    Pseudo                          MD5                                 Wavefunction (eV)    Charge density (eV)
---------  ------------------------------  --------------------------------  -------------------  ---------------------
Ag         Ag_ONCV_PBE-1.0.oncvpsp.upf     94f47bd0669c641108e45594df92fabc                 50.0                  200.0
Al         Al.pbe-n-kjpaw_psl.1.0.0.UPF    cfc449ca30b5f3223ec38ddd88ac046d                 30.0                  240.0
Ar         Ar_ONCV_PBE-1.1.oncvpsp.upf     46d28409cdd246843f76b7675277a949                 60.0                  240.0
As         As.pbe-n-rrkjus_psl.0.2.UPF     767315de957beeeb34f87d97bf945c8f                 35.0                  280.0

We should: