Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Register DevTools [http://github.com/JunoLab/DevTools.jl] #4250

Closed
wants to merge 1 commit into from

Conversation

MikeInnes
Copy link
Member

No description provided.

@tkelman
Copy link
Contributor

tkelman commented Dec 18, 2015

@tkelman
Copy link
Contributor

tkelman commented Dec 18, 2015

This is also a rather generic package name. Would have no idea this has anything at all to do with plotting or graphics from the name DevTools.

@MikeInnes
Copy link
Member Author

"DisplayPlotsWithElectronAndAlsoAProfiler.jl"?

@tkelman
Copy link
Contributor

tkelman commented Dec 19, 2015

ElectronPane or ElectronWindow or ElectronView or something like that? Can this use an existing system browser, or would that have to change inside Blink.jl?

@MikeInnes
Copy link
Member Author

This just calls Blink to produce windows, so yeah, it would have to change inside Blink.

This package provides graphical tools for Julia developers, currently including a means of displaying graphical output at the REPL as well as a profile viewer. I'm not sure how "ElectronPane" gets that across more clearly.

@tkelman
Copy link
Contributor

tkelman commented Dec 19, 2015

Lots of julia development is non-graphical. It's a common request to have Gadfly use a dedicated window for plots, which it looks like this does. Saying "Install DevTools" to make that possible isn't really intuitive to me. BlinkExtensions? Is there a technical reason the profiler piece and the plotting piece are together in the same package? (Separate BlinkPlots and BlinkProfileView packages would be clearer if they aren't sharing too much code.) Or that these aren't implemented in Blink itself?

@MikeInnes
Copy link
Member Author

Or that these aren't implemented in Blink itself?

For the same reason that ProfileView.jl and Immerse.jl aren't implemented inside Gtk.jl?

Separating the functionality and naming them separately could work, though it sounds like it also means doubling the number of name bikesheds we have to go through.

@timholy
Copy link
Member

timholy commented Dec 19, 2015

JunoDevGUIs?

@ViralBShah
Copy link
Member

I must admit that I found DevTools.jl to be a confusing name, and couldn't fathom what it had to do with displaying plots in windows, until @shashi told me that it was referring to chrome dev tools. I think ElectronPane is not great either. How about BlinkPlots and BlinkProfileView?

I really wish conditional modules work and can be precompiled, so that all this can be hidden from the users.

@ViralBShah
Copy link
Member

I think JunoDevGuis is not a good name, because the primary use case (at least my current usage) is without Juno. I love it that I can get a plot window and ProfileView from the terminal, without going through too much installation effort - and that it is cross-platform.

@ViralBShah ViralBShah closed this Dec 22, 2015
@ViralBShah ViralBShah reopened this Dec 22, 2015
@StefanKarpinski
Copy link
Member

using ChromeDevTools?

@ViralBShah
Copy link
Member

I was thinking about ChromeDevTools too.

@EricForgy
Copy link
Contributor

As I read down the comments, I was also thinking "ChromeDevTools".

Edit: Alternative names:

  • Chromium.jl
  • ChromiumDevTools.jl
  • Electron.jl (probably better for Blink.jl or a close cousin)
  • ElectronDevTools.jl
  • BlinkDevTools.jl

Of the above, I think ElectronDevTools.jl is the most accurate.

@Ismael-VC
Copy link
Contributor

What about JunoDevTools.jl ❔ Sounds nice and is very explicit to me.

@kmsquire
Copy link
Member

Bump

@musm
Copy link
Contributor

musm commented Apr 1, 2017

bump

@aviks
Copy link
Member

aviks commented Aug 23, 2017

Closing, since it's been a couple of years. Please do a new tag if necessary.

@aviks aviks closed this Aug 23, 2017
@tkelman tkelman deleted the omm/devtools branch August 23, 2017 23:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants