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

Rewrite of run_GUI and run_CMD #16

Open
MaximeBaudette opened this issue Oct 31, 2015 · 3 comments
Open

Rewrite of run_GUI and run_CMD #16

MaximeBaudette opened this issue Oct 31, 2015 · 3 comments

Comments

@MaximeBaudette
Copy link
Collaborator

I'm not very familiar with MatLab programming, but I think both interface can be rebuild around common core functions that both interface should use.
This way we could ensure a better maintenance of the code.

Also It seems that run_cmd is implemented in such a way that the user should modify the script for setting the parameters of his choice. I don't really like that and think it should be implemented in such way that everything is put in argument (using objects defining the plugins parameter)

@MaximeBaudette
Copy link
Collaborator Author

From @lvanfretti

It is very important we have all examples with scripts.

Indeed, in the current implementation there is no real CLI in rapid, but scripts can be prepared to directly interact with the core and run the toolbox's algorithms.

In a first stage we can update all examples with a script to run them.
In a further evolution we should implement a CLI with public methods (that would also be used in the GUI), and deprecate the direct interaction with the core.

@MaximeBaudette
Copy link
Collaborator Author

I added an issue to track that we add a script in each example #29

@janlav
Copy link
Contributor

janlav commented Mar 1, 2016

rapid() can be called with an object or an struct with properties/fields that conforms to that a RaPIdObject.

A suggestion is that we rename/remove run_rapid_cmd.m (rename it to rapid_scripting_template.m or something).

Can we create a detailed checklist for this issue?

janlav added a commit to janlav/JanForkofiTesla_RaPId that referenced this issue Mar 2, 2016
Closes ALSETLab#35 issue

The commit on this branch have changes made to adress (a part of) issues ALSETLab#34 and ALSETLab#16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants