AnderGray / ProbabilityBoundsAnalysis.jl

Probability bounds analysis in Julia
Other
25 stars 1 forks source link

Clashes with `Plots.jl` #32

Open dpsanders opened 2 years ago

dpsanders commented 2 years ago

My default graphing library is Plots.jl, but PBA clashes with it:

julia> a = uniform(0, 1)
Pbox:     ~ uniform ( range=[0.0, 1.0], mean=0.5, var=0.083333)

julia> using Plots

julia> plot(a)
WARNING: both Plots and ProbabilityBoundsAnalysis export "plot"; uses of it in module Main must be qualified

I'm not sure what is actionable here, but it should at least be mentioned in the README and docs.

Part of the review for the JuliaCon proceedings: https://github.com/JuliaCon/proceedings-review/issues/96

dpsanders commented 2 years ago

It would be great to make plot recipes for Plots.jl. You could maybe detect which plotting package is available and define code accordingly.

(But all that is necessary for the review is to add a note about this issue.)

AnderGray commented 2 years ago

plot recipes is the ideal way to do this. However when I tried the plotting with Plots.jl, I couldn't quite get it to look the way I liked, so I'd prefer to stick with PyPlot for now. What I have done is remove PyPlot as a dependence, and used Requires.jl. So that if you wish to plot, you need to using PyPlot

I still need to make note of this in the Documentation.