Skip to content
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

Updated figures, descriptions for Driver Station pages #2309

Draft
wants to merge 40 commits into
base: main
Choose a base branch
from

Conversation

pjbuterbaugh
Copy link
Contributor

Closes #2267.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Fixes build error do to line changes in main

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@pjbuterbaugh
Copy link
Contributor Author

Finishing up figures for logging section.

@sciencewhiz
Copy link
Collaborator

In general, it would be better to replace the old pictures with the new pictures, rather then deleting the old file and adding a new file. This keeps from breaking links in other files (see failing checks), and also makes it easier to review.

AyushSagar16 and others added 25 commits September 17, 2023 07:53

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Update code formatting article after wpiformat update

* update RLI
Add same note for rsvg-convert as was there for MiKTeX
* fixed link in tutorial-intro.rst

* Fix :ref: in tutorial-intro.rst

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
The Accelerometer interface was deprecated in wpilibsuite/allwpilib#5445.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Update gyro docs

The Gyro interface was deprecated in wpilibsuite/allwpilib#5445.

* Fix title levels
* Remove old command based stuff

also fix some broken links I found

* add redirect
* Add c++ counterparts for java shuffleboard snippets

* Add frc:: to c++ examples

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Update TrapezoidProfile docs

* Update source/docs/software/advanced-controls/controllers/trapezoidal-profiles.rst

Co-authored-by: Tyler Veness <[email protected]>

---------

Co-authored-by: Tyler Veness <[email protected]>
@pjbuterbaugh
Copy link
Contributor Author

In general, it would be better to replace the old pictures with the new pictures, rather then deleting the old file and adding a new file. This keeps from breaking links in other files (see failing checks), and also makes it easier to review.

Didn't see this - will fix.

Copy link
Collaborator

Choose a reason for hiding this comment

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

There's some changes in the 2024 beta DS. This picture and maybe some others will need to be updated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will evaluate and determine what needs to be changed, thanks for the head's up.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you get a screenshot that has less CAN devices attached so that it still shows free memory and disk space?

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.

Parts of the Driver Station pages are out of date
9 participants