Skip to content

Exciting parser species file - #273

Closed
VonaCecilia wants to merge 43 commits into
developfrom
exciting-parser-species-file
Closed

VonaCecilia wants to merge 43 commits into
developfrom
exciting-parser-species-file

Conversation

@VonaCecilia

Copy link
Copy Markdown
Collaborator

No description provided.

VonaCecilia and others added 30 commits October 20, 2026 11:49
add_mapping_annotation(..., m_def=X) stored X in
Section.more['mapper_m_def'] — unkeyed state on a definition shared by
every parser. The mapping parser pops that slot for whichever parser
builds its mapper first and discards that parser's own polymorphic
annotations: importing the orca schema package was enough to strip
KSpace/SelfConsistency from any FHI-aims parse in a full-plugin process
(nomad parse, Oasis), with parse-order-dependent results.

The slot is redundant: the helper already re-targets the keyed
annotation onto the m_def subclass, which the polymorphic scan resolves
under the parser's own key. Verified orca DLPNO output unchanged and
FHI-aims KSpace restored regardless of import order.
…y (e.g. 'PBE', 'PBE0', 'B3LYP', 'SCAN+rVV10')
…understand if n_point should be the total one or the reduced one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants