-
Notifications
You must be signed in to change notification settings - Fork 33
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
Move to JuliaPlots #103
Comments
I'm certainly not an expert in PlotlyBase but I'd be in for maintaining. |
@BeastyBlacksmith, unfortunately I can't speak much more than "Hello World" in Julia. So the only help I can offer is finding more bugs, testing features and proofreading documentation ... |
@BeastyBlacksmith yes, I can help with the maintenance duties. |
Hey @etpinard and @BeastyBlacksmith In principle I'm ok with this... however I'm not convinced that a move will help with maintenance. I think there have only been about 12 commits to PlotlyJS.jl since the move to JuliaPlots two years ago. Would you be up for a staged approach where we try to get support maintaining here and -- if we do get strong support -- we can make the move at that time? |
My concern is less increasing contributors ( which of course is always nice, but as you point out, is also not something that will magically happen just because its moved ) than being part of an organization will increase the likelihood that it doesn't become unmaintained for whatever reason with no way to pass over than forking it. |
I have been dabbling with plotlybase for quite some time and would also be available to contribute if needed. For example I just had some issues today with some wrong counter for SubPlots and realized that the PR fix was merged 9 months ago but no patch release has happened since. |
@sglyon is the author of this package. He has invested a lot of time and energy into creating, updating and maintaining it over the years. In the last two years, he most likely had new commitments and lacked the time needed for maintenance. At the same time, we, the users, would like it to be updated at the same pace as plotly.py. I think it would be fair that if PlotlyBase moves to JuliaPlots, the Readme file should mention his name, just as the Readme file for Plots.jl displays the name of its creator, Tom Breloff: |
Just submitted two PRs together with @sjkelly, the first is already merged. @sglyon I understand if you don't like the moving of the package. You could consider adding some users as contributors and agree on how changes are submitted, e.g. only by feature branches. BTW, will anyone of you be at the JuliaCon? That could be a good place to meet and discuss. |
@hhaensel, great that there are some updates again to this package. And I plan to be at the JuliaCon, but can't promise it yet ... |
Hey @sglyon,
would you be okay with moving this to JuliaPlots also?
The text was updated successfully, but these errors were encountered: