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

set timeMin to modelRun list timeMin #506

Merged

Conversation

BryonLewis
Copy link
Contributor

The removal of the @update:timeRang from SideBar.vue meant that the slider was always going to utilize the UNIX epoch time. This uses the global state.timeMin. I think this was an oversight of mine when I was converting the very heavily integrated component system signaling (where it was doing some passing up/down of information) into more of a store system when I came on to the project.

I do understand that our Store is really overloaded right now and at some point should probably be refactored but that is an issue for another time.

@BryonLewis BryonLewis requested a review from floryst September 13, 2024 13:03
@floryst
Copy link
Contributor

floryst commented Sep 13, 2024

Does the same change need to happen to annotationViewer/SideBar.vue? There's also a timemin change there.

@BryonLewis
Copy link
Contributor Author

Does the same change need to happen to annotationViewer/SideBar.vue? There's also a timemin change there.

It was sitting in my commit list in VSCode (I guess I forgot to click on commit all). Thanks,

@BryonLewis BryonLewis merged commit f363af5 into refactor-loadModelRuns-to-store Sep 13, 2024
8 checks passed
@BryonLewis BryonLewis deleted the modelRun-store-timerange-update branch September 13, 2024 13:12
floryst added a commit that referenced this pull request Sep 13, 2024
* Refactor loadModelRuns out of ModelRunList.vue

- Rename loadModelRuns to queryModelRuns
- Rename checkDownloading to refreshModelRunDownloadingStatuses

* Call refreshModelRunDownloadingStatuses directly

* set timeMin to modelRun list timeMin (#506)

---------

Co-authored-by: Bryon Lewis <61746913+BryonLewis@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants