-
Notifications
You must be signed in to change notification settings - Fork 1
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
send the camera centric positions of where we think the nodes are #101
Conversation
@@ -1035,9 +1035,6 @@ private void updateTeleopDrivingTarget(boolean recalculateGridPosition) { | |||
} else { | |||
// We're on the opposite side as our alliance | |||
// Try to go to the pickup position | |||
var predictedPoseForPickup = robotTracker.getLatestPose().getTranslation().plus( | |||
robotTracker.getVelocity().times(0)); | |||
|
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.
was this just unused?
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.
Yeah
yScoringOffsets | ||
.get(isRedAlliance ? AllianceSide.RED : AllianceSide.BLUE) | ||
.get(i) | ||
.get(selectedPosition.getScoringDirection()); |
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.
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.
} | ||
|
||
return redYs; | ||
} |
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.
whats the deal with this change? a refactor?
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.
That's the method we now call to get all the y-cords of the scoring positions so we can show them
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.
One of the methods this calls was unused so it wasn't updated to take the position offsets into account
var point = transformToCameraSpace(nodeLinePoints.get(i)); | ||
nodeLinePointsToSend[i * 3] = point.getX(); | ||
nodeLinePointsToSend[i * 3 + 1] = point.getY(); | ||
nodeLinePointsToSend[i * 3 + 2] = point.getZ(); |
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.
can we not make a 2d array and flatten it? this code is fine, but ugly
No description provided.