Jupyter notebook eval #12
Closed
Conversation
This file contains 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
Adding monkey patching fix suggested on forums
Adding code to do waypoint visualizations
fix: correcting some typos in creating waypoints objects
Minor comment changes
Updates and error fixes to pid.py.
Fixed integral term in PID
…tudents who want to use different networks for each.
…tudents who want to use different networks for each.
Added site boolean for TL node
* Implemented simple red light classifier and interface to change the classifier * minor refactoring * updated the thresholds and modified the message name * updated review comments * updated thresholds for stopping and red light size * Update waypoint_updater.py Updated the comment in line with the implementation from 2 to 4 for number of stopping ahead wapoints
thejerrytan
requested review from
ser94mor,
pkorivi and
Karthikeya108
and removed request for
pkorivi
ser94mor
requested changes
Jan 23, 2019
|
This pull request introduces 1 alert when merging 83de649 into ed7da4b - view on LGTM.com new alerts:
Comment posted by LGTM.com |
thejerrytan
force-pushed
the
jupyter-notebook-eval
branch
from
83de649 to
5111001
Compare
this is what i did: in order to remove those files from commit history. Now to get the validation set on your own:
|
|
Ah, something bad happens with the diff. Could you create a new branch from master and put an |
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.
Added a jupyter notebook to visualize data, and do sanity check for trained models.
This is not ready to be merged yet.
I am including it first because it may be helpful to some of you who wants to evaluate your trained DL models.
Going to use it to generate images for README later also.
I think i did sth stupid, i commited Vatsal's dataset - TFrecord files into the repo, 266MB, ~ 1000 files. Let me know if this is bloating up your githistory and i can rewrite the githistory.