-
Notifications
You must be signed in to change notification settings - Fork 5k
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
os_path is not defined #6458
Comments
Is the file that you are attempting to rename in a hidden directory, or hidden itself? |
@krassowski , No nothing is hidden . I created new file and tried to rename it and got this error.I think this is the error with new roll back because i renamed the file in other env (older version of Jupyter notebook) . |
I could not reproduce on Linux. If you continue getting this error in the new environment, could you please provide a full path to the file that you are trying to rename and the new name you are trying to set? If the paths requires removal of potentially sensitive details, please confirm that the sanitised path is still sufficient to reproduce. Could you also check if this is also reproducible with nbclassic, this is by running |
Thanks @krassowski it is working fine , now . |
I reproduced this bug unfortunately on Mac. arguments for rename_file: self.root_dir: '/Users/win/work' In this case, is_hidden returns True! because is_hidden assumes absolute path, but passed relative path. inside_root is '.ipynb' and startwith('.') returns True... Lines 199 to 201 in 4ba07e1
|
Hi, how did you solve the problem? I also have this bug on my mac. |
Yes, this all confirms what I suspected; two options:
|
Thanks. I installed Jupyterlab in my environment and now it works. |
Facing same issue on tljh setup. How did you solve it by installing JupyterLab? |
Yes. It works for me.
zape ***@***.***>于2022年9月15日 周四上午9:40写道:
… Facing same issue on tljh setup. How did you solve it by installing
JupyterLab?
—
Reply to this email directly, view it on GitHub
<#6458 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOL3XV27U7MUDZRB776GI7LV6MRMFANCNFSM5ZWVXEOQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Hi! @xuhuizhou-vt please check discourse. |
I reproduced this bug by the following steps:
I fix this bug by the following steps:
I think this is not a "bug":
Warning:
|
I faced the same issue and solved it through change the folder name first. Actually, it seems like the error is raised if the folder name is 'notebooks' or includes it any way, though it's a sheer guess. |
@pai-djmoon Thanks for this - I had my notebooks in a folder called 'notebooks'. I simply did |
Thanks @pai-djmoon - moving notebooks to a new folder worked for me, too. My original folder was not named notebooks or anything similar, though, so I don't think the name of the old folder matters. |
Is there any reason why this is still open? |
It would be great to have it working in the first place, not to rename the original folder to make it work |
Bit of a brutal solution, but I fixed it using |
Describe the bug
A clear and concise description of what the bug is.
os_path is not defined in rename_file function (line 579).
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
File should be renamed
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
IPython : 8.4.0
ipykernel : 6.15.0
ipywidgets : not installed
jupyter_client : 7.3.4
jupyter_core : 4.10.0
jupyter_server : not installed
jupyterlab : not installed
nbclient : 0.6.4
nbconvert : 6.5.0
nbformat : 5.4.0
notebook : 6.4.12
qtconsole : not installed
traitlets : 5.3.0
Python 3.9.12
Additional context
Add any other context about the problem here.
It is asked on discourse too (rename_failed)
The text was updated successfully, but these errors were encountered: