Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [papyrus-rt-dev] Update the Model Import tool for Neon

Hi, Charles,

I don’t know how close we are.  With the Papyrus Components Discovery component, we already have some of the necessary infrastructure in place for automatic p2 installs.  I don’t expect that it should be difficult to reuse/refactor this.  Although, refactoring for reusability outside of Papyrus could be an issue given that Papyrus Neon has now past its API Freeze.

The more interesting question is intercepting the “open” action on RSA-RTE models.  Happily, they already are easily recognizable (emx file extension, numerous XML namespaces).  I can think of a few different ways to approach this, with Eclipse’s retargetable open action, Common Navigator’s open and double-click extensions, and even a clunky custom editor that just launches the import (and/or discovery) wizard.  It would need a bit of investigation to scope this out.

cW

On 11 April, 2016 at 10:52:25, charles+zeligsoft.com (charles@xxxxxxxxxxxxx) wrote:

Hi Christian,

Thanks for the feedback!

I really like your proposal for the import. The question then becomes: how close are we to having this available. Papyrus-RT 1.0 will be available at the end of June and we need a good user experience at that point.

As for the Welcome page, that is the current plan.

Sincerely,

Charles Rivet
Senior Product Manager
charles@xxxxxxxxxxxxx

On 2016.04.11, at 10:42 , Christian Damus <give.a.damus@xxxxxxxxx> wrote:

Hi, Charles,

See some replies in-line, below, on the technical questions.

Christian

On 11 April, 2016 at 10:26:47, charles+zeligsoft.com (charles@xxxxxxxxxxxxx) wrote:

———— 8< ———— 

A question:

  • Could this be handled through the installation process/tool? That his, having the default product includes the importer, but allowing knowledgeable be able to leave it optional? Can OOMPH provide this level of configurability?


The experience that I, as a user, would like for model interchange scenarios like this is automatic discovery of the required importer.  Say I’m an RSA-RTE user and I have a workspace already that contains a bunch of models.  When I open one of these models in Papyrus-RT, what I should see is that Papyrus-RT recognizes that there is content in this model (probably indicated by a foreign XML namespace declaration) that it doesn’t handle natively, but for which it has an importer that it can install via p2.  The tool then prompts me to say something like “To edit this, it needs to be imported into the Papyrus-RT format.  This requires additional software to be installed.  Proceed?”  and, on acceptance, the tool then automatically installs the import feature using the p2 APIs.  This experience can be expanded to cover all of the import scenarios for other source tools as they are implemented over time.

Oomph does something very like this in the setup authoring.  The Setup Editor knows about tasks that can be added to a setup model, for which tasks the model definitions and tooling support are not currently installed.  When you try to add one of these tasks to your setup model, the editor basically runs you through the scenario I described above.

In any case, I think that maybe “import” oughtn't really be an explicit step that users should have to invoke to work with their RSA-RTE models.  Double-clicking to open the model should just be redirected to the import wizard.


And a comment:

  • Related to this, we need a “Getting Started” link on the “Welcome” window, and this topic must be in there. However, long-time users of Eclipse may just power through that screen...


Hardened Eclipsers probably do power through this screen, but we do notice when it pops up again in an installation that we’ve been using for some time, to highlight new features.  Even if the Welcome view has been closed by the user, if an update installs a plug-in that adds new content to this screen, it will be shown when the workbench re-starts and actually highlights the new content with a yellow shadow thing.

So, I think it definitely is worth adding Papyrus-RT getting-started material to this screen.  You’ve already developed tutorials, Charles, that would lend themselves fairly readily to encoding in Eclipse UA’s tutorial format.



Sincerely,

Charles Rivet
Senior Product Manager; Papyrus-RT product manager
charles@xxxxxxxxxxxxx

On 2016.04.11, at 08:19 , Peter Cigéhn <peter.cigehn@xxxxxxxxx> wrote:

Hi,

Good questions. Regarding the builds I personally have no opinion about that, since I do not understand or have insight in the implications of the different choices.

When it comes to the features, I have actually been thinking about exactly the same.

I realized for example that I had missed the Papyrus_RT Core C++ Feature, which is not included in the top level Papyrus-RT Feature either. This makes perfect sense also, since the bringing in the specific language feature should be optional (depending on which languages we should have). We had a discussion in the mailing list a while ago, about exactly this, and I made a simple picture back then

<image.png>

I guess in the same way as we want to keep the C++ (or C or Java and so on) as optional features, we should also have the import tool a separate feature, not included in the top level Papyrus-RT Feature.

So I would propose to move the new import feature as separate feature as well. If you have the need to import old RSARTE models, then you need to check one additional feature (apart from the action/target language that you want to use).

/Peter Cigéhn

On 11 April 2016 at 14:08, LETAVERNIER Camille <Camille.LETAVERNIER@xxxxxx> wrote:

Hi,

 

I’ve pushed a new patch set to include the RSA-RTE importer in a separate feature. However, this raises some additional questions:

 

-          Currently, we have 3 builds and 3 features:

o   Profile

o   Core

o   Tooling

 

It seems that the RSA Importer makes sense in the Tooling build, but not in the tooling feature. Thus, the new patch set adds a new feature (RSA Importer) included in the Tooling build. Should we configure a separate build for this?

 

-          I’ve added the RSA Importer feature to the top-level Papyrus-RT Feature. Does that make sense as well? If the end-user doesn’t want to install the RSA Importer, then he needs to manually check Profile + Core + Tooling, without the RSA Importer. But the top-level feature is supposed to contain everything else, so... not sure what’s best

 

Regards,
Camille

 

De : papyrus-rt-dev-bounces@xxxxxxxxxxx [mailto:papyrus-rt-dev-bounces@xxxxxxxxxxx] De la part de LETAVERNIER Camille
Envoy
é : lundi 11 avril 2016 09:54
À : papyrus-rt developer discussions <papyrus-rt-dev@xxxxxxxxxxx>
Objet : [PROVENANCE INTERNET] Re: [papyrus-rt-dev] Update the Model Import tool for Neon

 

Hi Peter,

 

Ø  maybe we should put this into its own feature, instead of including it directly in the tooling feature

 

I have no opinion on this, so, whatever the Papyrus-RT team prefers :) I’ll add a separate feature

 

Ø  When the RSARTE specific import extension is installed, do you also have to install the RSA import feature from Papyrus? Or will the RSA import functionality included automatically?

 

The standard RSA Import from Papyrus is automatically installed when installing the Papyrus-RT Import (provided that the Papyrus Extra update site is available, so this depends on your deployment). The extension adds no extra UI, it works exactly in the same way (Except that it adds support for Papyrus-RT rather than discarding the profile and stereotypes). Additional UI might be required if e.g. the RSA-RTE Import needs specific import options. But so far, this didn’t seem to be a use case

 

Regards,
Camille

 

De :papyrus-rt-dev-bounces@xxxxxxxxxxx [mailto:papyrus-rt-dev-bounces@xxxxxxxxxxx] De la part de Peter Cigéhn
Envoyé : lundi 11 avril 2016 09:27
À : papyrus-rt developer discussions <papyrus-rt-dev@xxxxxxxxxxx>
Objet : Re: [papyrus-rt-dev] Update the Model Import tool for Neon

 

Hi Camille,

 

As I commented on the Gerrit change, maybe we should put this into its own feature, instead of including it directly in the tooling feature? Since the import tool is a rather "one-shot"-kind of functionality, I guess it would be good if it could be installed optionally for those that don't want/need it. The new import feature could still be included in the top level Papyrus-RT feature in the same way as profle, core and tooling is already.

 

For my understanding also: When the RSARTE specific import extension is installed, do you also have to install the RSA import feature from Papyrus? Or will the RSA import functionality included automatically?

 

In practice, how will it work from an end-user perspective? Will it be the different sets of import options in the import wizard, or do the RSARTE specific import just extends the existing RSA import function, in practice causing it to to be able to also import RSARTE models?

 

/Peter Cigéhn

 

On 7 April 2016 at 13:06, LETAVERNIER Camille <Camille.LETAVERNIER@xxxxxx> wrote:

Hi all,

 

A few months ago, the Model Import tool in Papyrus (Neon) has been refactored to support extensions. The UML-RT part has been removed from the Papyrus plug-in, and contributed to Papyrus-RT. The patch was waiting for the migration of Papyrus-RT to Neon, which is now complete (At least for the tooling part, which is affected by this patch)

 

I’ve rebased the patch to the latest master, so it’s now ready to be merged. Just a few remarks:

 

-          The patch contributes a new Tooling plug-in and the associated Test plug-in, in version 0.8.0. Papyrus-RT still seems to be in 0.7.2

o   That’s not an issue, but maybe we want to remain consistent

-          The patch only adds new plug-ins, and doesn’t change any existing file, except:

o   Tooling pom.xml (Addition of a new module)

o   Tooling Tests pom.xml (Addition of a new module)

o   Tooling Feature (Addition of a new plug-in)

 

So I don’t expect any (major) conflict with other contributions

 

483828: [Model Import] Extend the Papyrus Migration Tool to support UML-RT models

https://bugs.eclipse.org/bugs/show_bug.cgi?id=483828

 

62188: Bug 483828: [Model Import] Extend the Papyrus Migration Tool to support UML-RT models [Id2d75c74]

https://git.eclipse.org/r/#/c/62188/

 

Camille


_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

 

_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev


_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

_______________________________________________ 
papyrus-rt-dev mailing list 
papyrus-rt-dev@xxxxxxxxxxx 
To change your delivery options, retrieve your password, or unsubscribe from this list, visit 
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev 
_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev


_______________________________________________
papyrus-rt-dev mailing list
papyrus-rt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/papyrus-rt-dev

Back to the top