-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Improved documentation to help people check their understanding #3780
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3780 +/- ##
==========================================
- Coverage 81.86% 81.77% -0.10%
==========================================
Files 299 299
Lines 15113 15157 +44
==========================================
+ Hits 12373 12395 +22
- Misses 2740 2762 +22
Continue to review full report at Codecov.
|
This is absolutely great :) |
Sure, I can do it. I guess we can add the questions I had added in the Jupyter notebook to the documentation, (to check their understanding). What about the reading group itself, how can we organize it? If there is interest I could coordinate. I can create a Google form on this. What do you think @rusty1s? Also, feel free to change the possible readings, if you have something better. I have also thought of explaining how to read csv data and work with heterogeneous graphs, but I am not sure where to put it. Perhaps we should maintain somewhere a list of things we would like to see added too. |
Yes, sounds good :)
I have no particular preference. Please feel free to go ahead with whatever you feel is best. Nonetheless, let's try to first merge this PR and create an issue with things we would like to add. WDYT? |
I've included the questions in the main documentation. A possible syllabus for reading group might rather be found on https://docs.google.com/document/d/1OWiVCteFLBQ1c7r9EwprB74OQSp7ku_SytBoC84gB00/edit?usp=sharing |
This is great. Thanks a lot. I further fixed some styling issues and typos :) |
Initial commit of reading group syllabus.