Skip to content
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

notebooks #41

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

notebooks #41

wants to merge 9 commits into from

Conversation

crdanielbusch
Copy link
Contributor

@crdanielbusch crdanielbusch commented Aug 5, 2024

Description

Work in progress.

Checklist

Please confirm that this pull request has done the following:

  • Tests added
  • Documentation added (where applicable)

Notes

@lewisjared
Copy link
Collaborator

@crdanielbusch Take a look at jupytext (I think it is in our copier repo). Jupytext allows us to just check-in a .py file instead of a .ipynb file. The bonus is that the .py file is waaaay easier to see what has changed in the notebook and to deal with merge conflicts as it is plain text instead of a JSON file.

If you open the .py files in jupyter lab they will generate a corresponding .ipynb file and keep it in sync with the .py file so changes in one impact the other and vice versa. We can then gitignore *.ipynb and just track the built notebook.

@prayner
Copy link
Contributor

prayner commented Aug 5, 2024 via email

@lewisjared
Copy link
Collaborator

I could imagine that the text to speech for a JSON file with mixed images, HTML and code would be less than ideal 😬

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.

3 participants