Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update Jupytext Notebook url path#6961

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

Merged
jtpio merged 2 commits intojupyter:mainfromRRosio:jupytext_url
Jul 18, 2023
Merged

Conversation

@RRosio
Copy link
Collaborator

Fixes#6942
I believe this addresses the last point in the issue above, as when a Jupytext Notebook is opened it no longer adds the/edit route but rather/notebooks.

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branchRRosio/notebook/jupytext_url

@parmentelat
Copy link
Contributor

cool, thanks@RRosio to haven care of this, please let me know if/when this gets released so I can give it a try

@RRosioRRosio marked this pull request as ready for reviewJuly 6, 2023 19:07
@jtpio
Copy link
Member

@parmentelat were you able to try this change with the built assets for this PR?

@jtpiojtpio added the bug labelJul 17, 2023
(widgetName==='default'&&ext==='.ipynb')||
widgetName==='Notebook'
widgetName==='Notebook'||
widgetName==='Jupytext Notebook'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Or maybe we can check ifwidgetName contains the stringNotebook?

jupytext is a third-party extension and not officially supported, so not sure we should hardcode its special case here.

RRosio reacted with thumbs up emoji
@jtpiojtpio added this to the7.0 milestoneJul 17, 2023
@parmentelat
Copy link
Contributor

parmentelat commentedJul 17, 2023
edited
Loading

@jtpio

Hi; unfortunately no, I am mostly away from keyboard these days
plus, I am not completely comfortable testing unreleased code, as my fuzzy understanding of the build process sometimes leads me to wrong conclusions, and so I tend to stick topip installs as much as I can
So, as much as I'd love to see the issue solved, I'm afraid I am not going to be able to help, at least not for the uncoming release ofnotebook if that's still the plan

@jtpio
Copy link
Member

I tend to stick topip installs as much as I can

Normally the built artifact (wheel) can be installed just like a released version withpip install. There wouldn't be any major differences between unreleased and released code if this PR gets merged and a new release is made right after.

Likely the most important part would be to make sure it is tested in an isolated virtual environment.

@parmentelat
Copy link
Contributor

I made quick test

however with this setup, on matter how I open a jupytext notebook, I still see the/edit/ url

@RRosio
Copy link
CollaboratorAuthor

With the PR I observed that right-clickingOpen with and selectingJupytext Notebook opens the jupytext notebook with the/notebooks route rather than the/edit route. However when double-clicking the jupytext notebook indeed it goes on to open at the/edit route unlike the classic Notebook which after double-clicking opens a jupytext notebook under the/notebooks route.
Is this a matter of updating Notebook 7 so that the double-click behavior be what it was in classic Notebook?

@jtpio
Copy link
Member

Is this a matter of updating Notebook 7 so that the double-click behavior be what it was in classic Notebook?

Normally double clicking should open the document with its default factory. Without configuration this seems to still beEditor for a Jupytext Notebook:

image

But testing with this PR and using the user settings provided by@parmentelat in#6914 (comment) to override the default factory, double clicking seems to be opening the file with/notebooks:

jupytext-default-factory-settings.mp4
RRosio reacted with thumbs up emoji

@RRosio
Copy link
CollaboratorAuthor

But testing with this PR and using the user settings provided by@parmentelat in#6914 (comment) to override the default factory, double clicking seems to be opening the file with/notebooks

Ah okay, I was missing the user settings. Once using those settings, I too witness that same behavior.

Copy link
Member

@jtpiojtpio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Thanks@RRosio for looking into this issue!

The change looks good, and should already improve the case of Jupytext 👍

@jtpiojtpio merged commit34ca875 intojupyter:mainJul 18, 2023
@github-actionsgithub-actionsbot locked asresolvedand limited conversation to collaboratorsJul 19, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@jtpiojtpiojtpio approved these changes

Assignees

@RRosioRRosio

Projects

None yet

Milestone

7.0

Development

Successfully merging this pull request may close these issues.

opening jupytext notebook with a base_url is failing

3 participants

@RRosio@parmentelat@jtpio

[8]ページ先頭

©2009-2025 Movatter.jp