Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

Default Plot, Targets, And Selections - #92

Merged
O957 merged 12 commits into
mainfrom
75-default-plot-targets-and-selections
Jul 30, 2025
Merged

Default Plot, Targets, And Selections#92
O957 merged 12 commits into
mainfrom
75-default-plot-targets-and-selections

Conversation

@O957

@O957 O957 commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator

For the full scope of this PR, please refer to issue #75 .

Specifically, this PR adds:

  • A button called All underneath the model selectbox that selects all models.
  • A button called None underneath the model selectbox that de-selects all models.
  • The default for the model selectbox set to all models.
  • Edits to the Next and Previous buttons to abide better with PEP-008, i.e. "_single_leading_underscore: weak “internal use” indicator."
  • Target selection defaults to the first target, alphabetically.
  • [-] A function called _cut_observed_to_forecast that truncates the observations to be extra_weeks longer than the minimum forecast date.
  • The default for reference to be the most recent reference date.

Somewhat out-of-scope but still useful, this PR also adds:

  • Two functions target_selection_ui and model_selection_ui in place of model_and_target_selection_ui, to keep the code more organized and atomic; corresponding updates have been made in main.
  • Two functions location_selection_ui and reference_date_selection_ui in place of location_and_reference_data_ui, again to keep the code more organized and atomic; corresponding updates have been made in main.

@O957 O957 added this to the [July 28, August 8] milestone Jul 30, 2025
@O957 O957 self-assigned this Jul 30, 2025
@O957 O957 linked an issue Jul 30, 2025 that may be closed by this pull request
Comment thread hubverse_annotator/app.py
@O957

O957 commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator Author

Issue at present where the defaults for the models and reference date to do not work (as in, they are not selected) if the forecast file is uploaded after the observations file. If the forecast file is uploaded alone everything works as intended.

Fixed via b1da630

@O957
O957 marked this pull request as ready for review July 30, 2025 16:30
@O957
O957 requested a review from dylanhmorris as a code owner July 30, 2025 16:30
@O957
O957 requested a review from damonbayer July 30, 2025 16:30
Comment thread hubverse_annotator/app.py Outdated
Comment thread hubverse_annotator/app.py Outdated

@damonbayer damonbayer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AFg6K7h4fhy2!

@O957
O957 merged commit bfb129b into main Jul 30, 2025
2 checks passed
@O957
O957 deleted the 75-default-plot-targets-and-selections branch July 30, 2025 17:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default Plot, Targets, And Selections

2 participants