Skip to content

Support an explicit frozen key on external projectors - #97

Open
elinscott wants to merge 1 commit into
aiidateam:mainfrom
elinscott:external-projector-frozen-key
Open

Support an explicit frozen key on external projectors#97
elinscott wants to merge 1 commit into
aiidateam:mainfrom
elinscott:external-projector-frozen-key

Conversation

@elinscott

Copy link
Copy Markdown
Contributor

Problem

Whether an external projector is frozen can only be expressed through the alpha bookkeeping that is specific to Yuhao's projector-generation protocol via hydrogenic wavefunctions: get_frozen_list_ext freezes every entry whose alpha is "UPF" or missing.

However, there are many other ways one can generate external projectors, and currently they have to invent a non-UPF alpha just to opt out of freezing.

Changes

  • get_frozen_list_ext honours an explicit boolean frozen key on each projector entry, taking precedence over alpha.
  • Tables without the new key behave exactly as before, so existing workflows are unaffected.

Testing

New unit tests cover both conventions and their precedence, including a mixed table checking the frozen indexing.

Whether an external projector was frozen could only be expressed
through the alpha bookkeeping of one particular projector-generation
protocol, where a missing alpha means a pseudo-atomic orbital and is
frozen. Projector tables produced outside that protocol had to invent
a numeric alpha to opt out of freezing.

- get_frozen_list_ext now honours an explicit boolean frozen key on
  each projector, taking precedence over alpha.
- Tables without the new key behave exactly as before.
- Add tests covering both conventions and their precedence.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@elinscott

Copy link
Copy Markdown
Contributor Author

CI will fail until #95 merged

@elinscott

Copy link
Copy Markdown
Contributor Author

@qiaojunfeng @npaulish

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.

1 participant