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

Add UML for address space to prepare future tasks #764

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ratara
Copy link
Contributor

@ratara ratara commented Jan 12, 2022

Note: I had to drop the original PR because I had to rebase my branch with master first and got into some trouble. :-)

Content

After introducing type hinting to address space, I discovered some data type issues. I would like to introduce some design documentation about address space to enable a discussion how to solve/refactor the issues.

Step 1: Add UML overview about address space ( in progress )
Step 2: Lets discuss what we can do on basis of these UML diagrams.
Step 3: Merge UML overview into master
Step 4: add new branch and resolve the discussed issues

Plant UML:
I thought about how to document and although I work a lot with Enterprise Architect, this is not a good solution here. I chose Plant UML because:

  • It is a known free vscode extension that can be run without any license.
  • It works offline.
  • Changes to the documentation can be tracked in git because the UML is generated from a text file and we have no binary blob.

Open questions:

  • I am open for ideas where to store the UML content. My first idea is side-by-side with the code but this can confuse. Maybe a sub folder is better?

Any thoughts?

@ratara
Copy link
Contributor Author

ratara commented Jan 12, 2022

This is a snapshot of the current generated UML diagram. I post it to visualise the possible benefit. Please do not review the diagram quality. I am creating the big picture now and will split the information afterwards in smaller pieces.

grafik

@oroulet
Copy link
Member

oroulet commented Jan 12, 2022

The problem with UML documentation is that it can get very quickly outdated... but that software has been realitvely stable for 10 years so maybe it is fine

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.

2 participants