Skip to content

Commit bd32e30

Browse files
jimustafaJeromeCCP9
authored andcommitted
fix(docs): fix parameter tables and add pre-commit check
1 parent 911f605 commit bd32e30

4 files changed

Lines changed: 52 additions & 7 deletions

File tree

.pre-commit-config.yaml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,3 +36,15 @@ repos:
3636
files: (?x)^(
3737
docs/.*
3838
)$
39+
- repo: local
40+
hooks:
41+
- id: check-generated-parameter-tables
42+
name: check parameter table CSVs match parameters.xml
43+
entry: bash -c 'cd docs/docs/parameters && python3 generate-parameter-tables.py && git diff --exit-code -- "*.csv"'
44+
language: system
45+
pass_filenames: false
46+
files: (?x)^(
47+
docs/docs/parameters/parameters\.xml
48+
| docs/docs/parameters/generate-parameter-tables\.py
49+
| docs/docs/parameters/.*\.csv
50+
)$

docs/docs/parameters/parameters.xml

Lines changed: 38 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -894,6 +894,39 @@
894894
</dimensions>
895895
<required>False</required>
896896
</parameter>
897+
<parameter tool="w90">
898+
<name>explicit_kpath</name>
899+
<types>
900+
<type>R</type>
901+
<type>R</type>
902+
<type>R</type>
903+
</types>
904+
<description>Explicit k-point path for the interpolated band structure</description>
905+
<groups>
906+
<group>plot</group>
907+
</groups>
908+
<dimensions>
909+
<dim>*</dim>
910+
</dimensions>
911+
<required>False</required>
912+
</parameter>
913+
<parameter tool="w90">
914+
<name>explicit_kpath_labels</name>
915+
<types>
916+
<type>S</type>
917+
<type>R</type>
918+
<type>R</type>
919+
<type>R</type>
920+
</types>
921+
<description>Labels for the explicit k-point path</description>
922+
<groups>
923+
<group>plot</group>
924+
</groups>
925+
<dimensions>
926+
<dim>*</dim>
927+
</dimensions>
928+
<required>False</required>
929+
</parameter>
897930
<parameter tool="w90">
898931
<name>bands_num_points</name>
899932
<type>I</type>
@@ -2250,39 +2283,39 @@
22502283
</groups>
22512284
</parameter>
22522285
<parameter tool="postw90">
2253-
<name optional_prefix="boltz_">smr_type</name>
2286+
<name optional_prefix="boltz_dos_">smr_type</name>
22542287
<type>S</type>
22552288
<description>Smearing type for the DOS</description>
22562289
<groups>
22572290
<group>boltzwann</group>
22582291
</groups>
22592292
</parameter>
22602293
<parameter tool="postw90">
2261-
<name optional_prefix="boltz_">adpt_smr</name>
2294+
<name optional_prefix="boltz_dos_">adpt_smr</name>
22622295
<type>L</type>
22632296
<description>Use adaptive smearing for the DOS</description>
22642297
<groups>
22652298
<group>boltzwann</group>
22662299
</groups>
22672300
</parameter>
22682301
<parameter tool="postw90">
2269-
<name optional_prefix="boltz_">adpt_smr_fac</name>
2302+
<name optional_prefix="boltz_dos_">adpt_smr_fac</name>
22702303
<type>R</type>
22712304
<description>Adaptive smearing prefactor</description>
22722305
<groups>
22732306
<group>boltzwann</group>
22742307
</groups>
22752308
</parameter>
22762309
<parameter tool="postw90">
2277-
<name optional_prefix="boltz_">adpt_smr_max</name>
2310+
<name optional_prefix="boltz_dos_">adpt_smr_max</name>
22782311
<type units="eV">P</type>
22792312
<description>Maximum allowed value for the adaptive energy smearing</description>
22802313
<groups>
22812314
<group>boltzwann</group>
22822315
</groups>
22832316
</parameter>
22842317
<parameter tool="postw90">
2285-
<name optional_prefix="boltz_">fixed_en_width</name>
2318+
<name optional_prefix="boltz_dos_">smr_fixed_en_width</name>
22862319
<type units="eV">P</type>
22872320
<description>Energy smearing (if non-adaptive) for the DOS</description>
22882321
<groups>

docs/docs/parameters/postw90-boltzwann-parameters.csv

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ boltz_dos_energy_step,P,"Step for the DOS (eV)"
2121
[boltz_dos_]adpt_smr,L,"Use adaptive smearing for the DOS"
2222
[boltz_dos_]adpt_smr_fac,R,"Adaptive smearing prefactor"
2323
[boltz_dos_]adpt_smr_max,P,"Maximum allowed value for the adaptive energy smearing (eV)"
24-
[boltz_dos_smr_]fixed_en_width,P,"Energy smearing (if non-adaptive) for the DOS (eV)"
24+
[boltz_dos_]smr_fixed_en_width,P,"Energy smearing (if non-adaptive) for the DOS (eV)"
2525
boltz_bandshift,L,"Rigid bandshift of the conduction bands"
2626
boltz_bandshift_firstband,I,"Index of the first band to shift"
2727
boltz_bandshift_energyshift,P,"Energy shift of the conduction bands (eV)"

docs/docs/parameters/w90-plot-parameters.csv

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ wannier_plot_spinor_phase,L,"Include the “phase” when plotting spinor WF"
1111
bands_plot,L,"Plot interpolated band structure"
1212
kpoint_path,"S,R,R,R,S,R,R,R","K-point path for the interpolated band structure"
1313
explicit_kpath,"R,R,R","Explicit k-point path for the interpolated band structure"
14-
explicit_kpath_lables,"S,R,R,R","Labels for the explicit k-point path"
14+
explicit_kpath_labels,"S,R,R,R","Labels for the explicit k-point path"
1515
bands_num_points,I,"Number of points along the first section of the k-point path"
1616
bands_plot_format,S,"File format in which to plot the interpolated bands"
1717
bands_plot_project,I,"WF to project the band structure onto"

0 commit comments

Comments
 (0)