-
Notifications
You must be signed in to change notification settings - Fork 8
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
Restore changes and tried data100 and data8 for classes #83
Restore changes and tried data100 and data8 for classes #83
Conversation
…rseid.py. Follow the README.md to know how to run this new file
…atching output metadata
… and data100 for classes
… 100. add it the terminal command and expected output in the Readme file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good on the first pass, we can go over the tooling in more detail tomorrow
@@ -2395,7 +2395,7 @@ | |||
"name": "python", | |||
"nbconvert_exporter": "python", | |||
"pygments_lexer": "ipython3", | |||
"version": "3.10.6" | |||
"version": "3.11.9" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
revert this to the prior version pls ty
@jlucarga a couple of quick things and i believe we can merge:
after that we can merge! |
…n. The name for main was updated and README.md file too with this new information
Run the following command line to test if you still getting an error for data8 and data100:
python refactor_testing_query_courseid.py 2242 data8 data100 compsci189
I will keep editing to print the same lines as the original query_courseid.py and add Shane's edit recommendations