Skip to content
This repository has been archived by the owner on Dec 27, 2021. It is now read-only.

Evaluate the need to use the XText/Sirius bridge from Altran #59

Open
mypsycho opened this issue Aug 30, 2021 · 2 comments
Open

Evaluate the need to use the XText/Sirius bridge from Altran #59

mypsycho opened this issue Aug 30, 2021 · 2 comments

Comments

@mypsycho
Copy link
Collaborator

mypsycho commented Aug 30, 2021

Altran developed a bridge to integrate Sirius and Xtext.

This bridges solves some issues.
We must check that POOSL also deals with those.
Otherwise we can integrate it.

For example, as model has no xmi:id maybe renaming model element may cause diagram corruption.

@mypsycho mypsycho added help wanted Extra attention is needed refactoring labels Aug 30, 2021
@wpiers wpiers added this to the M3 milestone Sep 6, 2021
@wpiers wpiers added P2 P1 and removed P2 labels Oct 13, 2021
@mypsycho mypsycho removed the help wanted Extra attention is needed label Nov 4, 2021
@mypsycho
Copy link
Collaborator Author

mypsycho commented Nov 5, 2021

As each poosl text work on it own ResourceSet, it could be interesting to have an AIRD for each file (instead of 2 per project).
With such mechanism, user could control when a resource is loaded and avoid conflict.

@mypsycho
Copy link
Collaborator Author

mypsycho commented Nov 5, 2021

Ideas to improve xtext/sirius interconnection:

  • limit conflict with 1 AIRD/file : Session is loaded and unloaded on demand (representation opening) instead of being always loaded. Not all conflict are avoided but at least user has a way to do.
  • provide a custom fragmentProvider in XtextResource: the provider use 'name' instead of list index for containment will limit refactoring impact.
    • Fragment should encode name + index to maximize recognition ?
    • Illegal construction would (no name or name conflict) would still corrupt diagram.
    • Provide a tool to fix representation ? when a representation target is not found, provide a reconciler.
  • Use properties view in Sirius

@wpiers wpiers removed this from the M3 milestone Nov 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants