kaizen-ai / kaizenflow

KaizenFlow is a framework for Bayesian reasoning and AI/ML stream computing
GNU General Public License v3.0
110 stars 76 forks source link

Convert Design Philosophy gDoc to Markdown #399

Closed DanilYachmenev closed 1 year ago

DanilYachmenev commented 1 year ago

FYI @samarth9008

Dylan-Celli commented 1 year ago

Minor bug noticed with > ../dev_scripts/convert_docx_to_markdown.py --docx_file "Design Philosophy.docx" --md_file Design_Philosophy.docx Adding the quotes to make the docx file a literal string causes the command to fail, the command worked after renaming the file to Design_Philosophy.docx and not putting the filename in quotes.

Dylan-Celli commented 1 year ago

When I used the dev_scripts/convert_docx_to_markdown.py it added a new line after every bullet point. I looked at the other md files and saw that they did not have the new lines after every bullet point so I removed the new lines. Should I have kept the new lines in?

Additionally, after converting all the hyperlinks had an add <span> html tag attached to them, the links still worked and the tag did not appear in the md preview but I removed the tags because I thought they were unnecessary. Should I add these back in?

Finally, the link associated with Measure seven times, cut once (Russian proverb) goes to a google doc that the reader does not have access to.

samarth9008 commented 1 year ago

When I used the dev_scripts/convert_docx_to_markdown.py it added a new line after every bullet point. I looked at the other md files and saw that they did not have the new lines after every bullet point so I removed the new lines. Should I have kept the new lines in?

No. We don't want extra space.

Additionally, after converting all the hyperlinks had an add <span> html tag attached to them, the links still worked and the tag did not appear in the md preview but I removed the tags because I thought they were unnecessary. Should I add these back in?

If the links are working than its fine.

Finally, the link associated with Measure seven times, cut once (Russian proverb) goes to a google doc that the reader does not have access to.

Thats fine.

yiyunlei commented 1 year ago

Additionally, after converting all the hyperlinks had an add <span> html tag attached to them, the links still worked and the tag did not appear in the md preview but I removed the tags because I thought they were unnecessary. Should I add these back in?

This HTML tag exists because the original Gdoc actually has an underline under the link, and GitHub doesn't render this underline, so you can both delete it or keep it.

yiyunlei commented 1 year ago

Minor bug noticed with > ../dev_scripts/convert_docx_to_markdown.py --docx_file "Design Philosophy.docx" --md_file Design_Philosophy.docx Adding the quotes to make the docx file a literal string causes the command to fail, the command worked after renaming the file to Design_Philosophy.docx and not putting the filename in quotes.

When I used the dev_scripts/convert_docx_to_markdown.py it added a new line after every bullet point. I looked at the other md files and saw that they did not have the new lines after every bullet point so I removed the new lines. Should I have kept the new lines in?

Thanks for pointing those out. I'll update the document accordingly.

yiyunlei commented 1 year ago

The filename can be put in quotes, but it can't contain any spaces.

Dylan-Celli commented 1 year ago

The link associated with the text "these slides" under ## Tips from a pro is no longer valid. Where can I find the the slides so I can fix the links? broken link: https://static.googleusercontent.com/media/research.google.com/en/people/jeff/stanford-295-talk.pdf

I tried looking up the slides and found Jordan Dean's Stanford page but it seems that the lecture slides link is broken there as well. Stanford page: https://talks.stanford.edu/jeff-dean-tensorflow-overview-and-future-directions/

samarth9008 commented 1 year ago

Just use this link for now. Mention about this in your PR.