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

CLI Strict Mode toggle needs to be created #1428

Open
CodeByDrescher opened this issue Jan 27, 2025 · 0 comments
Open

CLI Strict Mode toggle needs to be created #1428

CodeByDrescher opened this issue Jan 27, 2025 · 0 comments
Labels
Error Reporting All issues related to the relay of errors java Pull requests that update Java code SED-ML Tech Debt validation

Comments

@CodeByDrescher
Copy link
Contributor

While VCell CLI is separate from VCell Client, they share an underlying component of VCell Core. This means that some code that was originally designed with a GUI in mind ended up in Core; nothing involving the GUI directly, just features that make sense in a GUI environment.

This poses a problem with CLI, specifically: failure tolerance.

Consider: Importing SedML.

Desired GUI Behavior: Error messages and warnings when an archive is imported, but we want to keep as many parts of the model as possible. The user then has the option to fix the broken parts of the model using the GUI, rather than manually editing the archive to be VCell compliant.

Desired CLI Behavior: Immediately reject the archive if there's any error that would leave the import of the archive in an incomplete state. Since no editing can be done in CLI at this time, it is necessary to outright reject problematic archives, so that the user is not mislead with "complete" results.

To work around this issue, any part of VCell Core that we want to behave differently, depending on whether the entrypoint is GUI or CLI, should have a "strict mode" flag that can be set by CLI to enforce strict validation of run archives.

@CodeByDrescher CodeByDrescher added Error Reporting All issues related to the relay of errors java Pull requests that update Java code SED-ML Tech Debt validation labels Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Error Reporting All issues related to the relay of errors java Pull requests that update Java code SED-ML Tech Debt validation
Projects
Status: No status
Development

No branches or pull requests

1 participant