munich-quantum-toolkit / munich-quantum-toolkit/ddsim
Clean up the Simulator Hierarchy
Open
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 162
- Forks
- 40
- Avg merge
- 4h 49m
- Merged PRs (30d)
- 40
Description
Currently, the simulator hierarchy reflects it organic/chaotic growth. There are a few things to clean up:
- Check where the inheretence line could be improved (e.g.
StochasticNoiseSimulator) - Remove unnecessary members of
Simulator(e.g.rootEdgeand maybe the dd package (multipe simulators have to create their own packages anyway)
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 mapping the Simulator hierarchy, including StochasticNoiseSimulator, and identify where the inheritance line and shared members are defined. Review the use of rootEdge and the dd package across simulators; done requires an agreed hierarchy and removal of members or packages that are not needed, but the issue names no files or tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100