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

Initial tests with a Makie extension #47

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Initial tests with a Makie extension #47

wants to merge 5 commits into from

Conversation

trulsf
Copy link
Member

@trulsf trulsf commented Nov 3, 2024

This is just some initial testing with a possible Makie extension for plotting profiles (see also issue #16).

The following is tested and seems to work:

  1. A type recipe that handles periods and profiles as input and converts it to 2D points
  2. A full recipe that creates a new plot type for profiles
  3. Implementing specific plots by defining new methods for Makie.plot()

I guess there is a lot of room for experimenting if we should include this kind of support.

@trulsf trulsf requested review from hellemo and JulStraus November 3, 2024 21:17
@trulsf
Copy link
Member Author

trulsf commented Nov 3, 2024

@hellemo
Copy link
Member

hellemo commented Nov 20, 2024

Very cool!

Looking at some of the example plots, I think it would be really nice to hook into the mapping of AlgebraOfGraphics to visualize things like scenarios and strategic periods, such that one may easily switch between using facet plots or differentiating by color or linestyle or even do percentile bands. No idea how hard it would be though, I guess we would need to provide some form of tabular interface to the profiles.

@trulsf
Copy link
Member Author

trulsf commented Nov 22, 2024

I have added some initial testing with AOG using vectors of named tuples to implement the Tables interface for profiles. I kind of like this approach better than providing specific Makie plots.

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