-
Notifications
You must be signed in to change notification settings - Fork 10
UI for schedule and ship config building #193
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
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
f021258
add plan command to click
j-atkins b27698a
first draft of UI app
j-atkins ed245c9
updates before valid entry checking implemented
j-atkins a863ae7
starting to introduce more precise error messages
j-atkins 0b8f8f1
add textual as dependency
j-atkins 74f69dc
working towards better validation of inputs
j-atkins e171667
launch TUI in the browser
j-atkins 4abdac4
launch automatically to browser
j-atkins ac6af8f
validation via textual Input arguments working but with bugs
j-atkins 5be6ab4
instrument config refactoring
j-atkins 15d15b5
refarctoring and update config save_changes method
j-atkins 26d0ecd
improved error handling
j-atkins ee26959
Refactor error handling and validation; introduce UserError for user-…
j-atkins 1f7475f
fix potential bug in longitude validation
j-atkins 22b3691
add validators to space time region inputs
j-atkins 64e3486
Update save_changes method in ScheduleEditor
j-atkins 7e5ff85
Refactor error handling to provide detailed unexpected error messages…
j-atkins c7097e0
Refactor TODO comments for clarity and organization; update unexpecte…
j-atkins 3670252
remove reading in instrument selection from Excel for schedule
j-atkins 286f9f7
add .verify() methods to Save logic, remove browser launching, update…
j-atkins 1b22634
auto fill start and end times in SpaceTimeRegion if left blank in UI
j-atkins 111b410
Update TODO comments; enhance user messaging in schedule configuratio…
j-atkins b5eb357
add tests for UI; add pytest-asyncio dependency
j-atkins 7e8775a
update tests to reflect changes to how users select instruments
j-atkins a0ef9fd
tidy up `plan` command docstring
j-atkins 71bdd2d
tidy up
j-atkins 4512df0
post-review edits and fixes
j-atkins 9ca37ba
update test for new schedule error messaging
j-atkins be20ce8
tidy up old TODOs
j-atkins 1ee1f83
catch no matches error
j-atkins File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,6 +35,7 @@ dependencies = [ | |
| "PyYAML", | ||
| "copernicusmarine >= 2", | ||
| "yaspin", | ||
| "textual", | ||
| ] | ||
|
|
||
| [project.urls] | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.