Skip to content

Save notebooks as markdown#443

Merged
nllong merged 2 commits intodevelop-v2from
Save-notebooks-as-markdown
Apr 13, 2022
Merged

Save notebooks as markdown#443
nllong merged 2 commits intodevelop-v2from
Save-notebooks-as-markdown

Conversation

@haneslinger
Copy link
Copy Markdown
Contributor

@haneslinger haneslinger commented Apr 5, 2022

Any background context you want to provide?

What does this PR do?

We are looking for a better way to handle notebook diffs. Markdowns function just like .ipynb but dif better. I would like a way to make the project save all notebooks as markdowns by default but I can't figure out how. that said, simply changing the file extension does the job.

I referenced this blog post found by nick.

How should this be manually tested?

What are the relevant tickets?

#106

Screenshots (if appropriate)

@haneslinger haneslinger added Non-breaking Change Schema: No Changes No updates to the BuildingSync Schema Schema: General General update to BuildingSync labels Apr 6, 2022
@laurenliz22 laurenliz22 requested a review from nllong April 12, 2022 17:36
Copy link
Copy Markdown
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great!

For posterity sake, we need to launch jupyter lab, then open the markdown file with Jupytext Notebook.

image

I was concerned that outputs were not going to persist in markdown, but it does look like printed results do, which is what we need for this project.

image

@nllong nllong merged commit 713c411 into develop-v2 Apr 13, 2022
@nllong nllong deleted the Save-notebooks-as-markdown branch April 13, 2022 17:47
@nllong nllong removed the Schema: General General update to BuildingSync label Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Non-breaking Change Schema: No Changes No updates to the BuildingSync Schema

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants