-
Notifications
You must be signed in to change notification settings - Fork 75
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
Dsm 72 patch 1 #132
base: main
Are you sure you want to change the base?
Dsm 72 patch 1 #132
Conversation
…into branch-pred
wrapped the code from this branch into a utility class for running it
Pull Request Test Coverage Report for Build 4489613964
💛 - Coveralls |
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.
Wow, would you look at that!!!
I'd recommend an ipynb for the tutorial rather than a .py but hey I'm just a random guy on the internet with admin privileges.
Looks like you gotta run |
@scottgigante check tutorial. Someone else should go through and test this |
Still failing lints. Also brach_prediction.py -> branch |
@scottgigante what do I run to deal with lints? You want me to rename the file? |
Yes, there's a typo in the file name. |
Branch prediction wrapped as something reusable. Faithfully recreates example that was in
branch-pred
branch