- Notifications
You must be signed in to change notification settings - Fork5.5k
Use hatch backend#6425
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Use hatch backend#6425
Uh oh!
There was an error while loading.Please reload this page.
Conversation
blink1073 commentedMay 15, 2022
We're blocked bypypa/hatch#243. |
blink1073 commentedMay 15, 2022
Found a workaround using |
ofek commentedMay 18, 2022
FYI just releasedhttps://github.com/pypa/hatch/releases/tag/hatchling-v1.0.0 Also I think you can copy what you did hereipython/ipyparallel@e3e896a |
blink1073 commentedMay 18, 2022
Thanks, and congratulations! |
blink1073 commentedMay 18, 2022
We can remove |
ofek commentedMay 18, 2022
It's undocumented until I find an auto-generator for non-Click CLIs, but if build deps are met you can do |
blink1073 commentedMay 18, 2022
Thanks! Perhapshttps://sphinx-argparse.readthedocs.io/en/stable/? |
blink1073 commentedMay 18, 2022
Ah, you are using |
blink1073 commentedMay 18, 2022
@jtpio, any idea why the snapshot tests are failing? |
jtpio commentedMay 18, 2022
blink1073 commentedMay 18, 2022
Thanks! |
blink1073 commentedMay 18, 2022
Hmm, after adding the schemas the tests don't appear to start at all. |
jtpio commentedMay 20, 2022
hmm strange. For the UI tests the package is first installed with the following: Will check locally. |
jtpio commentedMay 20, 2022
add flake8 configadd artifactscleanupuse tbump to get current versioncleanup for new dep versionsclean up workflowsswitch back to bumpversionfix verionfixupfixupfixupfixupfix version checkfixupstry version handling againversion cleanupfix typescript errorundo bump2version changesinclude schemas and add build timeoutsfix workflow syntaxmore workflow cleanupclean up config
jtpio commentedMay 31, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Ah I thought CI was already uploading the built assets as artifacts. Maybe we could add the following step to the check release workflow so it's easier to inspect the sdist and wheel? -name:Upload Distributionsuses:actions/upload-artifact@v2with:name:notebook-jupyter-releaser-dist-${{ github.run_number }}path:.jupyter_releaser_checkout/dist |
blink1073 commentedMay 31, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Good call. I also played with my wip comparisonscript and got this output now:
|
blink1073 commentedMay 31, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Here's the latest output: |
jtpio commentedJun 1, 2022
Looking good! |
jtpio commentedJun 1, 2022
The Binder for this PR fails to build because of the following error: Which seems to be related to the |
jtpio commentedJun 1, 2022
This is likely to be fixed byjupyterlab/jupyterlab#12606. |
blink1073 commentedJun 1, 2022
Yeah I think we need to use a |
blink1073 commentedJun 1, 2022
Drat, now we have the issue that symlinks can't be created over existing files. With |
ofek commentedJun 1, 2022
Hatchling resolves symlinks fyi |
blink1073 commentedJun 1, 2022
The issue is the JupyterLab build command is trying to create symlinks over the files that are placed into |
blink1073 commentedJun 10, 2022
Ah, the symlink error was actually from a file in this repository, the binder is working now. I actually think this PR is good to go now. |
| source ="code" | ||
| [tool.hatch.build.targets.wheel.shared-data] | ||
| "notebook/labextension" ="share/jupyter/labextensions/@jupyter-notebook/lab-extension" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@blink1073 I'm not an expert on the filesystem layout for JupyterLab extensions, but should this be
| "notebook/labextension" ="share/jupyter/labextensions/@jupyter-notebook/lab-extension" | |
| "notebook/labextension" ="share/jupyter/labextensions/@jupyter-notebook" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks like this should be kept since it maps to"notebook/labextension" on the left hand side?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Yes, I agree
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Hmm, I could be wrong but I thought the output directorycontents were supposed to end up inshare/jupyter/labextensions/@jupyter-notebook/
If you look e.g. at the wheel forjupyterlab-katex, the contents of/jupyterlab_katex-3.3.0.data/data/share/jupyter/labextensions/@jupyterlab/katex-extension/ is
schemasstaticinstall.jsonpackage.jsonwhereas in notebook's wheel we currently have the following files in/notebook-7.0.0a4.data/data/share/jupyter/labextensions/@jupyter-notebook/:
lab-extensionE.g. this isjupyterlab-myst which works:https://github.com/agoose77/jupyterlab-myst/blob/320b15187ee605b1f948619e9e1f9cfa672eaf00/pyproject.toml#L57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The extension is nested in our case, under@jupyter-notebook. The full extension name is@jupyter-notebook/lab-extension.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Right, I'm with you. This is where the fact that/ is allowed in NPM package names threw me for a second!
| @@ -1,4 +0,0 @@ | |||
| { | |||
| "LabApp": {"collaborative":true,"expose_app_in_browser":true }, | |||
| "JupyterNotebookApp": {"collaborative":true,"expose_app_in_browser":true } | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Probably we want to keep this file, so RTC can be tested on Binder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Restored
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks!!
Looking good on Binder and also locally using the assets built by the releaser.
Left a small comment about the config for testing RTC on Binder, but otherwise looks good!
jtpio commentedJun 13, 2022
Starting a new pre-release with this change. |
jtpio commentedJun 13, 2022
Ah looks like something is wrong with the Python version:#6448 |


Uh oh!
There was an error while loading.Please reload this page.
setuptools/jupyter-packaging