Skip to content

Checkpointing update#350

Merged
grallewellyn merged 6 commits intodevelopfrom
checkpointing-update
Dec 4, 2023
Merged

Checkpointing update#350
grallewellyn merged 6 commits intodevelopfrom
checkpointing-update

Conversation

@grallewellyn
Copy link
Member

Added a note that the user must manually save in order to create a checkpoint of their notebook. This is the best location I could find to add this information.

@grallewellyn grallewellyn requested a review from rtapella October 18, 2023 15:51
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@wildintellect
Copy link
Collaborator

Related, maybe doesn't need to be in this PR. We need to remind users to exclude/ignore checkpoints in their .gitignore files as it bloats git unnecessarily.

.ipynb_checkpoints
*/.ipynb_checkpoints/*

@wildintellect
Copy link
Collaborator

gitignore is mentioned in a55dec1 we should link to that section. @grallewellyn that's all that's left to get this approved.

@grallewellyn
Copy link
Member Author

grallewellyn commented Dec 4, 2023

Is it good now? I thought this made more sense than linking an entire section

@wildintellect
Copy link
Collaborator

Looks better, it probably needs a link to somewhere that explains gitignore, I expect users to have never used before.

@grallewellyn
Copy link
Member Author

Okay, added link to explain gitignore

@grallewellyn grallewellyn merged commit 98f8259 into develop Dec 4, 2023
@grallewellyn grallewellyn deleted the checkpointing-update branch December 4, 2023 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants