Skip to content

Comments

Auto Shoot No Turret#92

Open
txrnqt wants to merge 24 commits intomainfrom
auto-shoot-final-maybe
Open

Auto Shoot No Turret#92
txrnqt wants to merge 24 commits intomainfrom
auto-shoot-final-maybe

Conversation

@txrnqt
Copy link
Member

@txrnqt txrnqt commented Feb 20, 2026

No description provided.

@txrnqt txrnqt requested a review from wilsonwatson February 20, 2026 05:50
@txrnqt txrnqt self-assigned this Feb 20, 2026
@txrnqt txrnqt marked this pull request as ready for review February 20, 2026 17:34
@wilsonwatson
Copy link
Member

Can you make a command to go to a specific distance from the hub? It'll help with getting data for #93.

io.setRightHopperVoltage(-1);
});

return retractSlowly.until(limitSwitchTouched).andThen(stop());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wilson stopped using the limit switch, so i'd remove that. and use the moto positions. and split the command into each side with their own position.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wdym by split? If its two commands, only one can run at a time.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

u are right about that. ya but if not using the PID there's no way to have it stop when each side are at the right position unless we're not concerned with that in this scenario?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is just to put compression on the balls so they move into spindexer I don't think that using pid is necessary

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree. The issue is of the right side goes faster than left, th switch will stop it and the left may not finish. Or vice versa, the left will continue to try to drive into the hard stop waiting for the right.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did my new push fix it?

double totalVelocity = baselineVelocity / Math.cos(Math.toRadians(baseline.hoodAngle));
double targetHorizFromHood = baselineVelocity * hoodFactor;
double ratio = MathUtil.clamp(targetHorizFromHood / totalVelocity, 0.0, 1.0);
double adjustedHoodAngle = Math.toDegrees(Math.acos(ratio));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hood angle != fuel exit angle

/* Controllers */
public final CommandXboxController driver =
new CommandXboxController(Constants.DriverControls.controllerId);
public final CommandPS5Controller driver =
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs to switch back before we run in shop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants