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

Barplot doesn't use bar height for limits #2182

Closed
Moelf opened this issue Jul 26, 2022 · 3 comments
Closed

Barplot doesn't use bar height for limits #2182

Moelf opened this issue Jul 26, 2022 · 3 comments

Comments

@Moelf
Copy link
Contributor

Moelf commented Jul 26, 2022

using WGLMakie

begin
	N = 10
	δ = 0.05
	cs = repeat([1,2], N÷2)
	heights = rand(N) * 10
	meshscatter(Point3f.(rand(N), rand(N), 0.00001rand(N)); 
	  	color = cs, 
		markersize = Vec3f.(δ, δ, heights),
		colormap = :Spectral_7,
		marker = Rect3f(Vec3f(0), Vec3f(1)),
	    shading=false
	)
end

image

begin
	N = 10
	δ = 0.05
	cs = repeat([1,2], N÷2)
	heights = rand(N) * 10
	meshscatter(Point3f.(rand(N), rand(N), 0.00001rand(N)); 
	  	color = cs, 
		markersize = Vec3f.(δ, δ, heights),
		colormap = :Spectral_7,
		marker = Rect3f(Vec3f(0), Vec3f(1)),
	 	axis = (type = Axis3,
            xlabel = "x label", ylabel = "y label", zlabel = "z label",
		),
	    shading=false
	)
end

image

@Moelf
Copy link
Contributor Author

Moelf commented Jul 26, 2022

something funky with Z auto limiting, adding

		    limits = (nothing, nothing, nothing, nothing, 0, 10)

solved it:
image

@ffreyer
Copy link
Collaborator

ffreyer commented Jul 26, 2022

This has been a thing forever. data_limits only considers the positions passed, not the extent of the meshes. This is also true for scatter.

@SimonDanisch SimonDanisch changed the title Axis3 is broken Barplot doesn't use bar height for limits Dec 29, 2022
@SimonDanisch
Copy link
Member

I think we should close this in favor of #2881

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants