-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
AIP-38 | Show historic metrics in Dashboard #42700
Comments
@bbovenzi I can take this up if you are okay. I have prototyped below versions as per design without the checkbox and the bar charts which don't have data from API. I can raise this as PR for the base part and fine tune the UI/UX during review or subsequent PRs. |
Nice! One nit, let's keep the phrase "Task Instances". |
@bbovenzi I am building the asset events section in the dashboard. As part of the dashboard designs the asset events was also to the right of historical metrics so my question is do they belong to the same section and are affected by the time range filter selected? If I select last 1 hour does historical metrics along with the asset events display the last 1 hour metrics only? |
Yeah, I guess they would be. I'll be honest though, I think the dashboard view is subject to a lot of change from what the designs are now and what we will have when 3.0 comes out. |
Once #42629 is merged, we can initialize our new homepage dashboard.
Right now
historical_metrics
only includes total counts. So we can build this part of the Dashboard mockups except for the bar graphs:That includes:
We'll ignore dag run type for now.
The text was updated successfully, but these errors were encountered: