-
Notifications
You must be signed in to change notification settings - Fork 29
ST6RI-682 Implemented loading of models from repository #616
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
base: master
Are you sure you want to change the base?
Conversation
- added command to list repository contents - added command to download and index repository project - refactored code
- Introduces a new SysMLv2 menu for all contributions - Renames repository upload feature - Command framework should check for the existence of repository descriptor for showing the command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Instead of
%publications
and%download
use%projects
and%load
. - The
%load
command (and utility) should have an option to use the project ID instead of the name.
- changed %publications to %projects - changed %download to %load - added optional arguments to %load --id=<ID> --name=<NAME> - changed SysMLRepositoryLoad util arguments -n, -name to reference project by name -id to reference project by id
@seidewitz as for the utility project name is now passed with |
@TheKorpos
In my experience, most open source tools use the GNU convention for command line options is to use single hyphen like |
@hpdekoning
There is a I'm not sure that allowing single hyphen long-names is a huge issue as long as it also accepts the GNU argument syntax. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general this looks fine, but it is possible that some comment added to #633 might be better handled here.
Allowing single-hyphen long-name options is OK, as long as double-hyphen is allowed, too. But the usage text should show them using double-hyphen, because that is the generally expected syntax. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Update the output from the
%projects
command so it has the following format:API base path: <base path URL> Project <project name> (<project UUID>) ...
- The
%load
command should provide clearer feedback on intermediate progress. - The returned success result from the
%load
command should have the form:Loaded Project <project name> (<project UUID>)
- I question whether we should include the
--branch
option as "experimental". Either we should include it and tests, with whatever limitations we currently have, or we should just not include it in this release.
I changed the output of
I changed the feedback. Now it only displays the two long running processes: the library UUID collection and the download/model creation. The UUID collection only happens on the first load so it's only displayed once when it happens. This is how the output looks:
The branch upload/download works with the pilot implementation but I'm not sure how much value it adds with the current limitations. Currently uploading to a branch (other than main) is basically the same as uploading to a seperate project. |
This PR adds basic model download capability using the standard API.
Overview
The model download functionality can be used
magic
commandsIn Jupyter, the downloaded models are stored in memory and can be used throughout the work session. The Eclipse and CLI versions serialize the model into separate XMI files (.sysmlx). In Eclipse IDE
.kerml
and.sysml
files are now added to the Xtext index so their content can be referenced using the textual syntax.Common changes
.sysmlx
and.kermlx
parsing, indexing:SysMLxRuntimeModule
,SysMLxStandaloneSetup
has been addedKerMLxRuntimeModule
,KerMLxStandaloneSetup
has been addedJupyter changes
%projects [-h][-help]
Lists all projects names and their ids
%load <project name> [-h][-help]
downloads the project and adds its contents to the xtext index.
%help
has been updatedEclipse changes
sysmlx
and.kermlx
.sysmlx
and.kermlx
resources are added to the xtext indexDownloads and saves models in
.sysmlx
.settings/org.omg.sysml.remote.properties
exists with the following contents:NOTE: the action shows up regardless for now. The file is needed to perform the operation.NOTE: a specific tester was implemented to hide the action from projects without the required properties file. However, this tester is not called before the sysml.ui plugin is activated, e.g. before a sysml editor is opened.
CLI app
org.omg.sysml.xtext.util.SysMLRepositoryLoadUtil
has been added-p, -project
: name of the project in the repository-l, -library
: location of the standard library-t, -target
: target location for saved resources-b, -base
(optional): URL of the repository, default: http://localhost:9000