NWChemEx / NWChemEx/PluginPlay
Add support for module namespaces
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 13
- Forks
- 1
- Avg merge
- 43m
- Merged PRs (30d)
- 4
Description
Over time it's likely that module collections will contribute modules with the same key. To disambiguate among the keys it would be nice to have namespaces. Like C++, the namespaces should support arbitrary nesting (it should suffice to concatenate the strings together into one namespace under the hood).
This would require modifying the way the ModuleManager stores modules. If we only require namespaces to be specified when there is ambiguity this should not break anything that currently exists.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the ModuleManager and the module collection code that stores and resolves module keys. Trace how keys are registered and looked up, then review existing tests for module retrieval. Done should include arbitrarily nested namespaces, disambiguation when keys collide, and unchanged behavior for existing unambiguous modules.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100