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

Wrong z-axis scaling due to not switching to 3D in scatter plot #622

Open
HorstBaerbel opened this issue Feb 28, 2023 · 8 comments
Open

Comments

@HorstBaerbel
Copy link

I have a dataset containing RGB888 colors in range [0,255]. When I choose a scatter plot and enable the z-axis the scaling is NOT like the x- y-axis scaling. I found the slider in settings ("Z scale proportion to Y") and set it to 1, but still the scaling is off.
grafik
grafik

@danmarshall
Copy link
Contributor

Thanks for reporting. I think there may be a bug for the Scatterplot chart where the z-axis mapping is enabled when it is not in 3d mode. You may just need to click the 3d icon in the top right. Here's screenshots where it's correct for the Column chart:
image
image

@HorstBaerbel
Copy link
Author

@danmarshall Thanks! That is a workaround, but I'll leave the bug open for now...

@HorstBaerbel HorstBaerbel changed the title Z-Axis scaling broken Wrong z-axis scaling due to not switching to 3D in scatter plot Feb 28, 2023
@danmarshall
Copy link
Contributor

Looking into is, the Z-axis is enabled in 2d mode purposefully - so that users can specify paint order of their scatterplots #512 . This is actually an important feature but seems to have caused some UI confusion that the z-axis is enabled. Perhaps it should be labeled better.

@HorstBaerbel
Copy link
Author

I was stumped by it. I did expect the graph to scale when setting the z-axis. Should the paint order not be a mapable field?

@danmarshall
Copy link
Contributor

@HorstBaerbel
Copy link
Author

Yes, I understand that. It is the drawing z-order for layering plots. Maybe change the label depending on 2D/3D?

@danmarshall
Copy link
Contributor

I think that's a good solution.

@HorstBaerbel
Copy link
Author

And thanks for the work! Must say I'm pretty new to SandDance, so I might make some mistakes and see everything from the "Dummy perspective" ;)

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

No branches or pull requests

2 participants