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

[WIP] Add 3d support to ggthemr #10

Merged
merged 2 commits into from
Nov 4, 2019
Merged

[WIP] Add 3d support to ggthemr #10

merged 2 commits into from
Nov 4, 2019

Conversation

asinghvi17
Copy link
Member

@asinghvi17 asinghvi17 commented Apr 10, 2019

Seems like this should be rather simple...

Edit: It seems it will not be quite as simple as envisioned.

TODOs:

  • Figure out how to transmit linestyle into the frame or grid of Axis3d
  • Make it look generally less ugly

@mkborregaard
Copy link
Collaborator

nice! could you post an example?

@asinghvi17
Copy link
Member Author

Will do, just testing at the moment.
It's kind of interesting, but the 3d linestyle and framestyle don't appear to be themable in the way that the 2d ones are...

@asinghvi17
Copy link
Member Author

using Makie, MakieThemes

AbstractPlotting.set_theme!(ggthemr(:fresh))

meshscatter(rand(10), rand(10), rand(10))

3d-fig1

@asinghvi17 asinghvi17 closed this Apr 10, 2019
@asinghvi17 asinghvi17 reopened this Apr 10, 2019
@asinghvi17
Copy link
Member Author

Colors looking good:

using Makie, MakieThemes

AbstractPlotting.set_theme!(ggthemr(:chalk))

meshscatter(rand(10), rand(10), rand(10), color = rand(10))

3d-fig2

@mkborregaard
Copy link
Collaborator

Yes the colors look nice. It's just the axes but I feel like the look of the 3d-axes isn't yet finished in Makie.

@asinghvi17
Copy link
Member Author

Billboarding would be really nice if we can get it, for the axis labels

(also these plots are really low resolution so that probably explains some of the ugliness)

@asinghvi17
Copy link
Member Author

Pinging @SimonDanisch, is there any way to modify grid attributes of Axis3d or will we need to attempt to add that as a feature in AbstractPlotting?

@asinghvi17 asinghvi17 self-assigned this Apr 10, 2019
@asinghvi17 asinghvi17 added the enhancement New feature or request label Apr 10, 2019
@asinghvi17
Copy link
Member Author

Per conversation over at JuliaPlots/AbstractPlotting.jl#115, it seems that linestyle will not be able to be transmitted without a rework of Axis3D's mechanism for generating Frames. Currently, the Frames are drawn as one linebuffer, so their linestyle can't be changed individually.

To be able to change the linestyle we can probably add a linestyle attribute to Axis3d whose type is Union{Nothing, Symbol}. That would globally change the linestyle for all of the grids (and the frames if not individually set) - it's a bit of a hack, but might work.

@asinghvi17
Copy link
Member Author

Here's the main issue - for full themability, we'll need to be able to customize how Axis3D is drawn. Is there any way to do that aside from overriding the method itself?

@mkborregaard
Copy link
Collaborator

Is this still wip?

@SimonDanisch
Copy link
Member

We could just merge this and improve it later?

@asinghvi17
Copy link
Member Author

Yeah, nothing else I can do here without changes in AbstractPlotting I think.

@mkborregaard mkborregaard merged commit e61208f into master Nov 4, 2019
@mkborregaard
Copy link
Collaborator

orsom

@SimonDanisch SimonDanisch deleted the as/3d branch November 4, 2019 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants