Skip to content

paths#27

Open
BrunoOsio wants to merge 11 commits into
masterfrom
paths-auto-nav
Open

paths#27
BrunoOsio wants to merge 11 commits into
masterfrom
paths-auto-nav

Conversation

@BrunoOsio
Copy link
Copy Markdown
Member

No description provided.


@Override
public void execute() {
this.intake.setSpeed(-1, 0);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

?


public void arcadeDrive(double forward, double rotation) {
this.drive.arcadeDrive(forward, -(rotation));
this.drive.arcadeDrive(forward, (rotation));
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

oq aconteceu?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

arrumei a tracao que tava invertido as curvas

}

public void setSpeed(double speedY, double speedX) {
this.motorLeft.set(ControlMode.PercentOutput, speedY);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

eu acho ok ter um speedY e X, ate mudaria os nomes dos motores pra motorY e X.

o release por ex é só o motor Y que tem q girar ne

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

da pra fazer uma sobrecarga desse método, em q tu passa o x e y e oq tu passa uma velocidade só... assim quando tu quiser iguais ou diferentes tu pode fazer

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

o problema que a gente tava enfrentando no intake eram esses dois parametros que estavam bugando o intake. Voltamos ao codigo antigo e funcionou

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

estranho bruno, acho que deve ter outra coisa por tras... talvez alguma coisa com os cans

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.

4 participants