Skip to content

Commit

Permalink
minor adjustments to Mesh and NumericalIntegration
Browse files Browse the repository at this point in the history
  • Loading branch information
EBB2675 committed Nov 19, 2024
1 parent 646430c commit cca109f
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 11 deletions.
4 changes: 2 additions & 2 deletions src/nomad_simulations/schema_packages/model_method.py
Original file line number Diff line number Diff line change
Expand Up @@ -1224,7 +1224,7 @@ def normalize(self, archive: 'EntryArchive', logger: 'BoundLogger') -> None:


class MolecularHamiltonianSubTerms(BaseModelMethod):
type=Quantity(
type = Quantity(
type=MEnum('coulomb', 'exchange'),
description="""
Typical sub-terms of the molecular hamiltonian.
Expand All @@ -1233,4 +1233,4 @@ class MolecularHamiltonianSubTerms(BaseModelMethod):
)

def normalize(self, archive: 'EntryArchive', logger: 'BoundLogger') -> None:
super().normalize(archive, logger)
super().normalize(archive, logger)
26 changes: 17 additions & 9 deletions src/nomad_simulations/schema_packages/numerical_settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,12 @@ class Mesh(ArchiveSection):
""",
)

spacing = Quantity(
type=np.float64,
shape=['dimensionality'],
description='Grid spacing for equidistant meshes. Ignored for other kinds.',
)

points = Quantity(
type=np.complex128,
shape=['n_points', 'dimensionality'],
Expand All @@ -117,14 +123,16 @@ class Mesh(ArchiveSection):
In the fixed grid methods, the number of angular grid points is predetermined for
ranges of radial grid points, while in the adaptive methods, the angular grid is adjusted
on-the-fly for each radial point according to some accuracy criterion.
"""
""",
)

def normalize(self, archive: 'EntryArchive', logger: 'BoundLogger') -> None:
super().normalize(archive, logger)

if self.dimensionality not in [1, 2, 3]:
logger.error('`dimensionality` meshes different than 1, 2, or 3 are not supported.')
logger.error(
'`dimensionality` meshes different than 1, 2, or 3 are not supported.'
)


class NumericalIntegration(NumericalSettings):
Expand All @@ -145,10 +153,10 @@ class NumericalIntegration(NumericalSettings):
"""

coordinate = Quantity(
type=MEnum('all', 'radial', 'angular'),
type=MEnum('full', 'radial', 'angular'),
description="""
Coordinate over which the integration is performed. `all` means the integration is performed in
all the space. `radial` and `angular` describe cases where the integration is performed for
Coordinate over which the integration is performed. `full` means the integration is performed in
entire space. `radial` and `angular` describe cases where the integration is performed for
functions which can be splitted into radial and angular distributions (e.g., orbital wavefunctions).
""",
)
Expand All @@ -158,16 +166,16 @@ class NumericalIntegration(NumericalSettings):
description="""
Integration rule used. This can be any 1D Gaussian quadrature rule or multi-dimensional `angular` rules,
e.g., Lebedev quadrature rule (see e.g., Becke, Chem. Phys. 88, 2547 (1988)).
"""
""",
)

weight_partitioning = Quantity(
weight_approximtion = Quantity(
type=str,
description="""
Approximation applied to the weight when doing the numerical integration.
See e.g., C. W. Murray, N. C. Handy
and G. J. Laming, Mol. Phys. 78, 997 (1993).
"""
""",
)

mesh = SubSection(sub_section=Mesh.m_def)
Expand Down Expand Up @@ -937,4 +945,4 @@ class GTOIntegralDecomposition(NumericalSettings):
)

def normalize(self, archive: 'EntryArchive', logger: 'BoundLogger') -> None:
super().normalize(archive, logger)
super().normalize(archive, logger)

1 comment on commit cca109f

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/nomad_simulations
   __init__.py4250%3–4
   _version.py11282%5–6
src/nomad_simulations/schema_packages
   __init__.py15287%39–41
   atoms_state.py1902189%13–15, 201–204, 228, 283–284, 352–353, 355, 537, 549–550, 611–615, 630–634, 641
   basis_set.py2502988%8–9, 122–133, 172–185, 244, 290, 471–475, 497–498, 542–545, 664, 695, 697
   general.py89891%4–7, 121, 185, 295–296, 306
   model_method.py2737971%10–12, 171–174, 177–184, 276–277, 297, 318–339, 355–381, 384–401, 587, 780, 791, 833–840, 878, 897, 977, 1034, 1109, 1223, 1236
   model_system.py3483789%45–51, 235, 254, 258, 261, 264, 290, 376–377, 454–455, 472–473, 686–689, 736–743, 917–918, 1140–1144, 1150–1151, 1159–1160, 1165, 1188
   numerical_settings.py2726476%12–14, 133, 184, 254, 256–257, 260–263, 267–268, 275–278, 287–290, 294–297, 299–302, 307–310, 316–319, 506–533, 608, 643–646, 670, 673, 718, 720–723, 727, 731, 778, 782–803, 858–859, 926, 948
   outputs.py1201092%9–10, 252–255, 295–298, 323, 325, 362, 381
   physical_property.py102793%20–22, 202, 331–333
   variables.py861286%8–10, 98, 121, 145, 167, 189, 211, 233, 256, 276
src/nomad_simulations/schema_packages/properties
   band_gap.py51590%8–10, 135–136
   band_structure.py1232580%9–11, 232–265, 278, 285, 321–322, 325, 372–373, 378
   energies.py42979%7–9, 36, 57, 82, 103, 119, 134
   fermi_surface.py17476%7–9, 40
   forces.py22673%7–9, 36, 56, 79
   greens_function.py991387%7–9, 210–211, 214, 235–236, 239, 260–261, 264, 400
   hopping_matrix.py29583%7–9, 58, 94
   permittivity.py48883%7–9, 97–105
   spectral_profile.py26012851%9–11, 57–60, 95–98, 199–300, 356–368, 393–396, 416, 421–424, 466–502, 526, 573–576, 592–593, 598–604
   thermodynamics.py752764%7–9, 35, 56, 72, 81, 90, 101, 110, 137, 147, 157, 172–174, 177, 193, 213–215, 218, 234, 254–256, 259
src/nomad_simulations/schema_packages/utils
   utils.py791680%8–11, 65–74, 83–84, 89, 92, 169–170
TOTAL261651980% 

Tests Skipped Failures Errors Time
402 0 💤 0 ❌ 0 🔥 5.819s ⏱️

Please sign in to comment.