MDANSE [RFC] Qvectors rework

Description of work Implement independent new scheme for handling Q-Vectors without shells.

Fixes Fixes #656 Fixes #458 Fixes #446 Fixes #705

To test New tests in UnitTests/NewQVectors

Mar 17 '25 16:03 oerc0122

Would you agree that the main reason for the RegisterFactory class is to register the subclasses explicitly and not automatically?

Mar 20 '25 08:03 MBartkowiakSTFC

Would you agree that the main reason for the RegisterFactory class is to register the subclasses explicitly and not automatically?

Yeah, it also means you can have multiple aliases (if desired) and you can have more than one class per file allowing sensible grouping.

Scanning files is not terribly safe and also means that it's picking up temporary files from my editor.

Mar 20 '25 08:03 oerc0122

Overall, the suggested changes are looking good. Especially with generators being generators, and the clear examples in the docstrings.

It looks promising in the context of issues #446 and #458 so definitely we should go ahead with these changes.

Mar 20 '25 09:03 MBartkowiakSTFC