-
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
Simultaneous Rover Programming #506
Conversation
@@ -18,7 +18,7 @@ var blocklyNodes = {}; | |||
var graphLines = {}; | |||
var loggerNodes = {}; | |||
var currentBlocklyNodeID = undefined; | |||
var blocklyExecuting = false; | |||
//var blocklyExecuting = false; |
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.
Remove commented out code.
One comment, then 👍 |
Now when we stop execution, it stops on the "current" rover, instead of stopping all of them at once. This now depends on virtual-world-framework/vwf#480. Please review! |
Be sure to do a rebase then 👍 |
…ode as a property of the blocklyNodes.
multiple rovers simultaneously, and with a UI that actually works (i.e. the reset button actually works when you choose a rover that is currently executing code).
stopAllExecution behavior.
calling stopExecutionForNode, which expects a node ID to be explicitly passed in.
4edab05
to
a29fc94
Compare
Discovered a bug after rebase. Please review my fix. |
@@ -73,7 +72,7 @@ vwf_view.firedEvent = function( nodeID, eventName, eventArgs ) { | |||
break; | |||
|
|||
case "blocklyStarted": | |||
startBlocklyButton.className = "reset"; | |||
//startBlocklyButton.className = "reset"; |
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.
Should this commented code be removed entirely?
One comment, but otherwise 👍 |
…cution Simultaneous Rover Programming
Remember to delete branches when you're done with them. GitHub gives the option after a merge. |
Rovers now work simultaneously, with some of Brett's suggestions incorporated to make the UI actually work with it.