Uploading most of Sarah's files as well as my own contributions#5
Merged
gwaybio merged 34 commits intoPennGCB:masterfrom Sep 21, 2017
sklasfeld:master
Merged
Uploading most of Sarah's files as well as my own contributions#5gwaybio merged 34 commits intoPennGCB:masterfrom sklasfeld:master
gwaybio merged 34 commits intoPennGCB:masterfrom
sklasfeld:master
Conversation
Member
|
This is comprehensive, thanks for getting the repo up to speed so quickly @sklasfeld Three general questions/comments:
|
Collaborator
Author
|
Member
Then I suggest scrapping all of the 2016 files - python 3 came out almost 10 years ago and many of the early concerns have been addressed. If 2.7 is still a large part of the course, then perhaps naming the files with py27 instead of 2016 would be more informative.
Yeah, that's what I mean - folders will help organization |
…zza posts for README files
Collaborator
Author
|
Let me know if that is okay. |
Member
|
LGTM - I like the renaming of the piazza posts to I will merge this in |
Collaborator
Author
|
Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sarah sent us her stuff at the beginning of the year. This is mostly everything. I also added the lectures that I edited.