Skip to content

Migrate UMLet to PlantUML#24

Closed
cr1sschwager wants to merge 40 commits intoeclipse-score:mainfrom
cr1sschwager:Migrate-UMLet-to-PlantUML
Closed

Migrate UMLet to PlantUML#24
cr1sschwager wants to merge 40 commits intoeclipse-score:mainfrom
cr1sschwager:Migrate-UMLet-to-PlantUML

Conversation

@cr1sschwager
Copy link
Copy Markdown

@cr1sschwager cr1sschwager commented Jun 20, 2025

Migrate UMLet to PlantUML Diagrams #20

Covered parts:
[x] configuration
[x ] events_fields
[x] extensions
[x] ipc_tracing
[x] partial restart
[x] runtime
[x] service_discovery
[x] shared_mem_layout
[x] skeleton_proxy

Procedure:

  • Conversion of the UMLet diagrams to PlanUML via Claude Sonnet 3.7/4
  • UMLet does not have a syntax and the LLM therefore has to interpret the picture
  • Remarks to the prompt: LLM tries to rebuild the PlantUML files not only from the input files, but also from the general knowledge and code base. Therefore we needed to forbid the usage of the code base to get 1:1 replicas of the diagrams
  • Afterwards manual review and finetuning (Human in the Loop)

cr1sschwager and others added 27 commits June 23, 2025 13:00
@cr1sschwager cr1sschwager force-pushed the Migrate-UMLet-to-PlantUML branch from 0c9a1bc to 69f47eb Compare June 23, 2025 11:03
@cr1sschwager cr1sschwager changed the title WIP: Migrate UMLet to PlantUML Migrate UMLet to PlantUML Jun 26, 2025
@cr1sschwager cr1sschwager marked this pull request as ready for review June 26, 2025 13:39
hoe-jo
hoe-jo previously requested changes Jul 2, 2025
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following Relations are missing:

  • Inheritance
  • Containment
  • knows
    Also Factory Classes are not modeled

Supposedly this is valid for all diagrams

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hoe-jo: Can you please point me to the explicitly missing parts. I compared the diagram once more with the input file and for me this is a 1:1 copy content-wise.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the late reply! I just took another look at the diagram and for some reason the links are there now. It must have been some misconfiguration in my setup of the rendering.

@hoe-jo hoe-jo dismissed their stale review July 9, 2025 07:24

Finding not correct

Copy link
Copy Markdown

@piotrchodorowski piotrchodorowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some diagrams like the ones pointed below that are very big and hard to follow. Is there any way that those can be decomposed into some smaller portions? Do you think it would make sense?

skeleton_proxy_binding_model.puml
generic_proxy_model.puml
shared_mem_layout_classdiagram.puml
proxy_event_state_machine_model.puml
lola_message_passing_model.puml
event_lola_model.puml
structural_view.puml

@LittleHuba
Copy link
Copy Markdown
Contributor

Thanks for the contribution. We took your work and split it in separate PRs #43 to #51.
I'll close this PR and will follow up on your work based on the new PRs.

@LittleHuba LittleHuba closed this Aug 4, 2025
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.

5 participants