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

added python notebook #2

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

added python notebook #2

wants to merge 6 commits into from

Conversation

amwink
Copy link

@amwink amwink commented Jan 19, 2024

for the survey BLOG post

@emdupre
Copy link
Member

emdupre commented Jan 19, 2024

Thanks, @amwink !

Just to confirm : I think this is all content that's currently inside the repository. Is the main change you're proposing adding a .ipynb rendering of the .md file ? If so, this is what the jupytext header on this file is for -- it automagically (😉 ) opens the file as a jupyter notebook in environments with jupyter installed, like My Binder.

If there's something else that I'm not seeing in reviewing these files, though, please let me know !

@amwink
Copy link
Author

amwink commented Jan 22, 2024

Ah no, my understanding of Binder was just that it imports a repository with .ipynb notebooks (so nothing would happen if your repository does not have a notebook). So the markdown file is 'jupitext'?

@emdupre
Copy link
Member

emdupre commented Jan 22, 2024

So the markdown file is 'jupitext'?

Yes, exactly ! Jupytext is useful generally for working with notebook files, especially under version control. I've set up the markdown file to be Myst Markdown, which has the added benefit that Binder can natively understand it. So, the screenshot I had shared of the running notebook was on Binder, using the repo as-is !

@amwink
Copy link
Author

amwink commented Jan 23, 2024

Understood. The notebook runs al the scripts from start (reading the csv) to finish (the p-values). If all those strips are in the .md then the ipynb is redundant.

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