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

revise dependencies, add dependency diagram #114

Open
5 tasks
VladimirAlexiev opened this issue Dec 19, 2024 · 0 comments
Open
5 tasks

revise dependencies, add dependency diagram #114

VladimirAlexiev opened this issue Dec 19, 2024 · 0 comments
Labels
enhancement New feature or request ontology

Comments

@VladimirAlexiev
Copy link
Contributor

VladimirAlexiev commented Dec 19, 2024

Here are the current module dependencies after taking into account #112:

  • can we eliminate the cyclical dependency between check_method and compliance_verification_report?
  • permitting_stages is not imported anywhere, maybe we should import it to aec3po?
  • are there other non-imported modules?
  • aec3po redundantly imports document, evidence, check_method. Maybe remove these imports?
    • On the other hand, the list of imports is the first thing one sees on the home page, so maybe it's better to list ALL modules there? I prefer this variant
  • Add a dependency diagram to the main doc https://ci.mines-stetienne.fr/aec3po/. The diagram is made with graphviz or plantuml; the source is in the first link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ontology
Projects
None yet
Development

No branches or pull requests

1 participant