Skip to content

Conversation

@frazane
Copy link
Contributor

@frazane frazane commented Oct 20, 2025

  • fixes the tick labels of the x (lead time) axis which were overlapping
  • adds interactivity: one can now zoom and move the x axis to focus on specific ranges of the lead time, and the range of the y axis will adapt to the data in each plot automatically
  • (extra) formatting

@frazane frazane changed the base branch from feat/decouple-inference-preparation-and-execution to main October 20, 2025 08:59
@frazane frazane marked this pull request as draft October 20, 2025 08:59
@jonasbhend
Copy link
Contributor

@frazane can we reduce this to be just about the vega-lite spec? I think this is fairly minimal and should be able to merge instantly. Please let me know if you want me to have a go (I'd just cherry-pick 866434e on top of main).

@frazane
Copy link
Contributor Author

frazane commented Oct 22, 2025

@frazane can we reduce this to be just about the vega-lite spec? I think this is fairly minimal and should be able to merge instantly. Please let me know if you want me to have a go (I'd just cherry-pick 866434e on top of main).

@jonasbhend this is branched off #68. Once that is merged, changes compared to main will be minimal. This is also why it's still marked as draft, I am waiting for the other PR.

@jonasbhend
Copy link
Contributor

I know, thanks. But this doesn't really depend on #68, right, so it could be decoupled (that being my point, but if we can get away without doing it by merging #68 first, all the better).

@frazane frazane force-pushed the feat/update-vega-spec branch from 866434e to 1a75163 Compare October 22, 2025 09:31
@frazane frazane marked this pull request as ready for review October 22, 2025 09:31
@frazane frazane merged commit 8ee5831 into main Oct 22, 2025
4 checks passed
@frazane frazane deleted the feat/update-vega-spec branch November 3, 2025 14:58
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.

3 participants