-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Migrating telamonian/jupyterlab-hdf to @jupyterlab/jupyterlab-hdf5 #10
Comments
This sounds good to me! I have had many people request an extension like this. |
Yum. I probably haven't requested the extension but I sure could use it. |
I'm in favor, I know there is a lot of interest in well-supported HDF extension. There has also been several false starts (at least one of them mine :) ) |
Can you say a bit about how it works? In particular, the problems we've faced before are that double-clicking in the filebrowser automatically requests the content of the file, and that the notebook server file api doesn't support asking for more specialized content (like a url to some server-side service). How does your extension get around these limitations of these notebook server and contents api? |
Process-wise, here are steps for incorporation into a Jupyter org: https://github.com/jupyter/governance/blob/master/newsubprojects.md#incorporation |
@jasongrout The builtin file handling stuff has a baked-in constraint that a file path point to a real, existing resource available on a local (to the server) filesystem. Since I wanted to be able to path into the directory-like
Integration of hdf5 into the browser (both the default filebrowser one and the hdf browser supplied by my extension) proved tricky, for two reasons:
For now, the problem of browser integration is "solved" via monkey patches (shoutout to @ian-r-rose) that enable the correct opening behavior when |
Integration of the HDF5 stuff with the data browser from |
Just for keeping this discussion on topic.
Could you please share some motivating factors for moving this to the jupyterlab org? Given that the extension already exists, I don't think arguments of the kind "I would like this to exist" are enough? Some standard pros and cons:
Another general point for discussion: Do we currently have a policy for which extensions to accept into the jupyterlab org? Do we welcome all extensions? Which criteria are needed to be accepted? How do we determine which extension go into the main repo, and which are kept separate. Hopefully if we answer these questions (in written form), it can be used as a guidance for other extension authors considering if their extension should be moved to the lab org/repo as well :) |
Some relevant points to the above questions from the Jupyter governance: Subprojects should:
|
This is my main reason for migrating. Currently the extension works, but its functionality is somewhat basic. As I start to add features I want to get as much feedback as possible so that I know what's of interest to the user-base. I also in general want more people to use it. There's also a bunch of good synergies between
|
The timeline of the incorporation process, according to the governance docs:
On Wednesday it will have been a week since I posted this proposal here. If no one has any objections, can we please pull the trigger on migrating |
I have opened #12 to write down our process for adding a new repo to the jupyterlab org. |
I did a run-through of the source repo, and I think it is ready to bring in now. I'd say let's give a day and if no one disagrees go ahead and bring it in. |
For the record, we had a long discussion this morning about adopting packages into the official JupyterLab github org. As I remember it, we went over the points @vidartf brought out above. We are exercising subjective judgement here for a pre-1.0 project that has generated good enthusiasm in the community and among core devs.
Max is taking lead for now, and interest is wide enough among other developers that we would be willing to help.
Several people in the community (representing larger organizations) have expressed broad interest. As mentioned elsewhere, there have been a few attempts at this already, and having a single extension combine efforts on an hdf5 reader is important.
Max said testing would be an important next step in this repo. Given that it is pre-1.0, we feel it can come in now and we can work on it in the jlab repo as an experimental pre-1.0 project.
We feel the future is bright here for this one.
It integrates with JupyterLab, and Max is working on PRs for JupyterLab to make it work much better.
Max has been an outstanding contributor demonstrating he understands what this means, and is willing to do so in this repo as well.
This project does have well-defined scope of an extension in JupyterLab providing hd5 reading capabilities.
Yes.
This was a definite yes from the discussion in the dev meeting today. |
We also decided to make the decision guidelines for this process more clear in #12. We also decided that someone besides Max would look over the code (@blink1073 did today). Once the repo is moved in, it then falls under the Jupyter governance model, and would need to:
|
I spoke to @telamonian today and we are going to move ahead with the migration now. |
+1 |
It is done: https://github.com/jupyterlab/jupyterlab-hdf5 |
Thanks again @telamonian! |
To be clear, the repo is in the jupyterlab org.
This is jupyterlab/jupyterlab-hdf5#13
This is jupyterlab/jupyterlab-hdf5#14
To my understanding, this is done. |
I finally got around to updating all of the licenses, and to adding more Jupyter people as maintainers of the I still haven't tried doing a release of the migrated |
@telamonian, once you accept the npm org invite I will add you as an owner. Before then you won't be able to create new packages under that org. |
|
Super cool! |
I would like to move telamonian/jupyter-hdf into the
@jupyterlab
org. That will involve:jupyterlab/jupyterlab-hdf5
github repo in the org@jupyterlab
org on npmjsI will be the primary maintainer of
jupyterlab/jupyterlab-hdf5
.The text was updated successfully, but these errors were encountered: