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

Feature/andy demo #10

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Feature/andy demo #10

wants to merge 17 commits into from

Conversation

andywong36
Copy link

Plotly dash app.

image

@andywong36 andywong36 requested review from brockf and jwdink November 30, 2022 20:27
@andywong36 andywong36 self-assigned this Nov 30, 2022
@brockf
Copy link

brockf commented Jan 25, 2023

I would love to expand on this a bit and merge in so that it's a core part of torchcast.

How difficult would it be to generalize the app so that it could be a tool that users could configure to explore/evaluate any set of torchcast forecasts?

@andywong36
Copy link
Author

andywong36 commented Jan 25, 2023

@brockf Right now, at any one time, you can look at
a. 2 sets of "Actuals"
b. one kind of forecast for the actuals.

Are you asking if we can have more than 2 actuals? or just spam the plot with all the forecasts?

Either or, it's easy.

If you meant something else, I'm not sure I understand

@brockf
Copy link

brockf commented Jan 25, 2023

At the highest level, my question is: how can we make this a more generic dashboard which can be configured for the user's problem? For example, right now it loads df = pd.read_csv(DATA_DIR / "df_electricity.csv.gz", parse_dates=["time"]), but that is highly specific for our electricity forecasting problem.

Could we modify the dashboard such that it takes as configuration (1) a forecasting dataset, and (2) some column metadata and then we could have a dashboard for any set of forecasts?

@andywong36
Copy link
Author

Yes we can 👍

If you have an example of other datasets to support, then I can write up something.

@uadnan uadnan force-pushed the main branch 2 times, most recently from f30b6a6 to ed64632 Compare October 9, 2024 14:22
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