Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor code for modularization #46

Draft
wants to merge 41 commits into
base: main
Choose a base branch
from

Conversation

heringerp
Copy link
Collaborator

@heringerp heringerp commented Oct 9, 2024

Large rewrite to allow the modular introduction of new analyses into panacus without having to touch many components.

This is still work in progress

Tasks:

  • Introduce data manager to handle graph data (allowing multiple analyses to reuse data)
  • Add Analysis trait which each new analysis has to implement
  • Create Analysis implementation for all existing subcommands
  • Hide inner workings from analyses
  • Add more tests
  • Restructure html generation

@heringerp
Copy link
Collaborator Author

Hi @lucaparmigiani, you were working on some major changes as well right? It might be beneficial if you could push these before I start making this PR merge-able, since there will likely be a lot of conflicts. However, for this PR there is still a lot of work to be done, so no need to hurry, I just wanted to give you a quick heads-up.

Best regards,
Peter

@heringerp heringerp marked this pull request as ready for review October 25, 2024 13:15
@heringerp heringerp marked this pull request as draft October 25, 2024 13:15
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