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

jupyter notebook #3

Open
wants to merge 2 commits into
base: master
from
Open

jupyter notebook #3

wants to merge 2 commits into from

Conversation

@incredibleray
Copy link

incredibleray commented Sep 4, 2020

No description provided.

@huangsam
Copy link
Owner

huangsam commented Sep 5, 2020

Thanks for the pull request @incredibleray! Do you intend to cover multiple lessons with syntax.ipynb? If not, I would prefer if you rename this content to variable.ipynb since it's currently not covering anything else at the moment. Happy to hear your thoughts on the matter.

@sonarcloud
Copy link

sonarcloud bot commented Sep 5, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@incredibleray
Copy link
Author

incredibleray commented Sep 5, 2020

Sam,

I forgot to drop a note here, this is a proof of concept, I am creating a small notebook file to see if you like this approach.

I think the file tab here does not really show the github rendering power, if you take a look at my forked repo here, you can see how it is rendered and also a link to open this notebook on colab and try it out right away without installing anything

@huangsam
Copy link
Owner

huangsam commented Sep 5, 2020

Gotcha thanks for giving me the heads up. I like the rendering on GitHub. And trying it out on Colaboratory seems easy enough. Just noticed that Colaboratory has GitHub integration.

@huangsam
Copy link
Owner

huangsam commented Sep 5, 2020

Have you considered converting Python files to Jupyter notebooks automatically? I found this resource on the APIs:

https://nbformat.readthedocs.io/en/latest/api.html#module-nbformat.v4

@huangsam huangsam self-requested a review Sep 5, 2020
@huangsam huangsam self-assigned this Sep 5, 2020
@huangsam huangsam removed their request for review Sep 5, 2020
@huangsam
Copy link
Owner

huangsam commented Sep 8, 2020

@incredibleray btw I just changed up my repository such that it has no more print statements. Then you can simply look at the assert statements and determine how to best visualize the content that way.

See #5 for more context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.