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

Add --blueprint to plot_dashboard_stress #6996

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

teh-cmc
Copy link
Member

@teh-cmc teh-cmc commented Jul 25, 2024

Adds a --blueprint flag to plot_dashboard_stress that will force it to send a blueprint that sets a neat 5s window query for all plots.

Blueprint(
    Grid(*[
        TimeSeriesView(
            name=p,
            origin=f"/{p}",
            time_ranges=VisibleTimeRange(
                "sim_time",
                start=TimeRangeBoundary.cursor_relative(offset=rr.TimeInt(seconds=-2.5)),
                end=TimeRangeBoundary.cursor_relative(offset=rr.TimeInt(seconds=2.5)),
            ),
        )
        for p in plot_paths
    ]),
    BlueprintPanel(state="collapsed"),
    SelectionPanel(state="collapsed"),
)

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested the web demo (if applicable):
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
  • If applicable, add a new check to the release checklist!
  • If have noted any breaking changes to the log API in CHANGELOG.md and the migration guide

To run all checks from main, comment on the PR with @rerun-bot full-check.

@teh-cmc teh-cmc changed the base branch from cmc/chunkified_queries_5_new_time_series to main July 25, 2024 16:49
@teh-cmc teh-cmc changed the base branch from main to cmc/chunkified_queries_5_new_time_series July 25, 2024 16:49
@teh-cmc teh-cmc changed the title TODO 6: todo Add --blueprint to plot_dashboard_stress Jul 25, 2024
@teh-cmc teh-cmc added 🔨 testing testing and benchmarks 🧑‍💻 dev experience developer experience (excluding CI) 🚀 performance Optimization, memory use, etc do-not-merge Do not merge this PR include in changelog labels Jul 25, 2024
@teh-cmc teh-cmc marked this pull request as ready for review July 25, 2024 16:52
@Wumpf Wumpf self-requested a review July 26, 2024 08:49
@teh-cmc teh-cmc force-pushed the cmc/chunkified_queries_5_new_time_series branch from 301c344 to 76c5d4a Compare July 29, 2024 12:35
@teh-cmc teh-cmc force-pushed the cmc/chunkified_queries_6_dashboard_blueprint branch from e4a5cac to 83074d4 Compare July 29, 2024 12:41
Base automatically changed from cmc/chunkified_queries_5_new_time_series to main July 29, 2024 16:03
@teh-cmc teh-cmc force-pushed the cmc/chunkified_queries_6_dashboard_blueprint branch from 83074d4 to 673632c Compare July 29, 2024 16:04
@teh-cmc teh-cmc removed the do-not-merge Do not merge this PR label Jul 29, 2024
@teh-cmc teh-cmc merged commit 69434fd into main Jul 29, 2024
30 of 31 checks passed
@teh-cmc teh-cmc deleted the cmc/chunkified_queries_6_dashboard_blueprint branch July 29, 2024 16:06
teh-cmc added a commit that referenced this pull request Jul 29, 2024
…6997)

Adds a `Transformables` subscriber that greatly improves
`TransformContext`'s performance by preventing unnecessary queries (see
comments in code for details).

Better name welcome.

- DNM: requires #6996

cc @Wumpf @jleibs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 dev experience developer experience (excluding CI) include in changelog 🚀 performance Optimization, memory use, etc 🔨 testing testing and benchmarks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants