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

Picto Xtext Example: add plantuml-based transformation and other views #120

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

alfonsodelavega
Copy link
Contributor

This PR adds a PlantUML-based transformation to visualise the entities of the Xtext's dmodel language:

  • The domain model language now has a string doc element for each entity, which is shown as a note in the views
  • Navigation between views and with the Xtext document has been maintained
  • Layers support has been added (add/remove reference labels, supertypes and subtypes, and so on)

Additional views:

  • A specific bar chart that counts datatypes, entities, features
  • A pinset-based table that counts elements of each type, and the number of times they are used in the model (e.g. how many times a datatype is used in an entity feature)

The previous Graphviz transformation has been maintained as well, as it is referenced in the Xtext article. That article will be updated to reflect these new visualisations.

@kolovos
Copy link
Contributor

kolovos commented Nov 22, 2024

Thanks for the PR! My suggestion would be to

  • Remove the Graphviz transformation as PlantUML is a more appropriate target notation for this visualisation
  • Make All Entities the default view to be shown when the Picto view opens
  • Move Readme, Stats and Types after the Entities folder

On a related note I had to upgrade the domainmodel.target target platform to use http://download.eclipse.org/releases/2024-09 for this to work on my MacBook M3 and also had to comment out <unit id="org.eclipse.sdk" version="0.0.0"/> as it couldn't get resolved (not sure why as this used to work). After doing that I was able to get this to work, but then when I select the Readme view, I get the following error:

Screenshot 2024-11-22 at 20 17 01

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