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

Surface plots have extra artifacts on macOS/GR 0.73 #550

Open
penelopeysm opened this issue Sep 8, 2024 · 0 comments
Open

Surface plots have extra artifacts on macOS/GR 0.73 #550

penelopeysm opened this issue Sep 8, 2024 · 0 comments

Comments

@penelopeysm
Copy link

penelopeysm commented Sep 8, 2024

Hello! The following surface plot (found in the Plots.jl test suite, the same one as in #507) has some spiky artifacts when run with GR 0.73 on macOS.

Other operating systems are fine.

Git bisect says that this began with the commit d391790. Given that this isn't really a change to the Julia codebase, is this something that should be reported even further upstream?

Code

using Plots

xs = collect(0.1:0.05:2.0)
ys = collect(0.2:0.1:2.0)

X = [x for x  xs for _  ys]
Y = [y for _  xs for y  ys]

Z = (x, y) -> 1 / x + y * x^2

surface(
    X,
    Y,
    Z.(X, Y),
    xlabel = "longer xlabel",
    ylabel = "longer ylabel",
    zlabel = "longer zlabel",
)

Expected

artifact 2

Actual

artifact

penelopeysm added a commit to penelopeysm/Plots.jl that referenced this issue Sep 9, 2024
 - Disabled reference tests 25 and 30, which require StatsPlots support,
   for all backends (in PlotsBase/src/examples.jl)
 - Separate tests that are skipped and tests that are known to be broken
   due to upstream issues
 - Re-enable reference test 41 as upstream issue has been fixed
   (JuliaLang/julia#47261)
 - Disable reference test 50 because of upstream issue
   (jheinen/GR.jl#550)
penelopeysm added a commit to penelopeysm/Plots.jl that referenced this issue Sep 9, 2024
 - Disabled reference tests 25 and 30, which require StatsPlots support,
   for all backends (in PlotsBase/src/examples.jl)
 - Separate tests that are skipped and tests that are known to be broken
   due to upstream issues
 - Re-enable reference test 41 as upstream issue has been fixed
   (JuliaLang/julia#47261)
 - Disable reference test 50 because of upstream issue
   (jheinen/GR.jl#550)
BeastyBlacksmith added a commit to JuliaPlots/Plots.jl that referenced this issue Sep 10, 2024
* CI fixes (#4974)

* Update tests for Latexify 0.16.5

Behaviour was changed such that ã becomes \tilde{a}, instead of
\textnormal{\~{a}}.

* Test Julia 1.6 on x86 macOS instead of ARM

See: https://discourse.julialang.org/t/how-to-fix-github-actions-ci-failures-with-julia-1-6-or-1-7-on-macos-latest-and-macos-14/117019

* Bump GR compat and Plots version number

* update plotly show methods for PlotlyKaleido v2

* remove Pkg

* add it back

---------

Co-authored-by: Simon Christ <christ@cell.uni-hannover.de>

* Don't need to edit CI for 1.6 as no longer supported

* Don't test GR/50 on macOS

* Reorganise failing tests

 - Disabled reference tests 25 and 30, which require StatsPlots support,
   for all backends (in PlotsBase/src/examples.jl)
 - Separate tests that are skipped and tests that are known to be broken
   due to upstream issues
 - Re-enable reference test 41 as upstream issue has been fixed
   (JuliaLang/julia#47261)
 - Disable reference test 50 because of upstream issue
   (jheinen/GR.jl#550)

---------

Co-authored-by: Simon Christ <christ@cell.uni-hannover.de>
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

1 participant