Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Draft] Create/edit run configuration #122

Closed
craig-willis opened this issue Jan 21, 2021 · 1 comment · Fixed by #205
Closed

[Draft] Create/edit run configuration #122

craig-willis opened this issue Jan 21, 2021 · 1 comment · Fixed by #205

Comments

@craig-willis
Copy link
Contributor

craig-willis commented Jan 21, 2021

Revised based on 6/14/21 dev call

Requirements:

  • A tale has a single default run configuration (see [Draft] New endpoints to manage recorded run configurations girder_wholetale#492)
  • The user can edit the run configuration (i.e., the entrypoint script, test script)
  • A tale can have multiple run configurations
  • The user can add, edit, rename, configurations The
  • A run configuration consists of a name, the path to an entrypoint script (required) and test script (optional). (Note: "type" in the mockups will not be supported and can be ignored).

Mockups:
Mockups for this feature are incomplete. There is one from @hategan's proof-of-concept and another partial mockup.

@hategan
Copy link

hategan commented Jun 14, 2021

  • Note: "type" in the mockups will not be supported and can be ignored.

Maybe this is referring to an initial iteration, but, in my opinion, supporting at least local and container are important to allow a decent development workflow. The idea being that "local" allows uses to quickly invoke an executable for testing purposes and rapid prototyping, but provides little guarantees of reproducibility, whereas "container" involves building a container from scratch, which is not suitable for rapid prototyping since it tends to take a long time, but essential for reproducibility.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants