-
Notifications
You must be signed in to change notification settings - Fork 391
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
Warning "File Changed" in JupyterLab after reloading notebook from disk #978
Comments
Hi @ser3n1ty87 , thank you for the detailed report! Well that is not expected, as you have reloaded the notebook you should not see that message. I'll try to reproduce this in a test (but I can't do this now, hopefully I'll have some time for this next week). |
Just a quick question, do you think you could take a note of the timestamps of each operations? I mean, timestamp of when you save the notebook in Jupyter (or timestamp of files on disk just after you do so), then the timestamp after the edit in Vim, and finally the timestamps when you get the warning in Jupyter when you try to save there? Is there any change if you exit Vim after you edit the |
Hi @mwouts, sorry for the delay...
There also isn't any difference if I close vim or not and I also don't have any autosave configured in vim. |
Thank you @ser3n1ty87 for taking the time to gathering all this information ! That is really helpful. I should be able to reproduce (and fix this) in a new test. I'll keep you posted. |
Hi @ser3n1ty87 , I have prepared a fix for this. Would you mind giving a try to the upcoming version? You can install it with
Please let me know if that solves the issue on your end. And thank you so much for reporting this! |
Hi @mwouts, I just tested your fix and the issue seems to be solved now. |
Thank you @ser3n1ty87 for the confirmation. I'll integrate the fix and deliver a new version. |
Hello, I'm getting a bug very similar to this when using 1.16.5. Saving any notebook even right after opening makes the "File Changed" warning appear. Downgrading to 1.16.4 fixes the issue. edit: I will open a new issue for visibility |
Hi!
I am using JupyterLab together with Jupytext (which is great btw) and paired one of my notebooks
my-notebook.ipynb
with a percent scriptmy-notebook.py
.When I am doing bigger changes and refactoring I am usually using Vim/Neovim for modifying the
my-notebook.py
directly.For getting those changes back into my
my-notebook.ipynb
file I am using "Reload Notebook from Disk" in JupyterLab.This also works as expected as I can see that the code part of my .ipynb notebook (in memory) is now equivalent to the content of the .py script.
However, whenever I now want to save the .ipynb notebook to disk I get the following warning:
Basically, it doesn't matter if I choose to revert or overwrite then as the .ipynb file gets correctly saved.
Even though this issue is only a little inconvenience, I am wondering: Is this to be expected or am I doing something wrong here?
May it be that this warning has something to do with the timestamp which the .ipynb notebook (in memory) "inherits" when reloading from disk w.r.t. the timestamps of the ipynb notebook (on disk) and the .py script (on disk)?
The text was updated successfully, but these errors were encountered: