-
Notifications
You must be signed in to change notification settings - Fork 1
Update README organization #38
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
base: main
Are you sure you want to change the base?
Conversation
| ## Table of Contents | ||
| 1. [Setup](#setup) | ||
| 1. [Linux](#linux) | ||
| 1. [Bare Metal](#bare-metal) |
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.
this link doesn't work (bare-metal)
| 2. [WSL](#wsl) | ||
| 3. [Docker](#docker-1) | ||
| 3. [macOS](#macos) | ||
| 1. [Docker](#docker-2) |
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.
this link doesn't work (docker-2)
README.md
Outdated
| If you've never contributed to a git repository before on this laptop or environment, you might receive an error message saying you don't have access. In that case visit [this tutorial](https://docs.github.com/en/authentication/connecting-to-github-with-ssh/about-ssh) to set up SSH for local GitHub access. | ||
| After cloning the code, we need to set up our IDE: VSCode. If you already have it, great. Otherwise follow [this](https://code.visualstudio.com/download) tutorial. We recommend installing the mypy and autoDocstring VSCode extensions. Our autoDocstring settings are `Docstring Format: Numpy` and `Start On New Line: Start docstring on new line`. | ||
| After cloning the repository build the workspace and install the dependencies. |
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.
missed comma, After cloning the repository, build....
README.md
Outdated
| ## First Time Setup | ||
| ### Get Ubuntu | ||
| To run our code you need an environment with Ubuntu 24. Select the guide below that fits your operating system. Some operating systems have multiple options. Here are our recommended installation options: |
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.
missing comma I think, To run out code, you need...
README.md
Outdated
| Restart. | ||
| Follow the instructions to [Clone the Repository](#clone-the-repository) and [Setup the IDE](#setup-the-ide) before continuing. |
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.
setup the ide link doesn't work
| - Press `F1` or `ctrl` `shift` `p` and choose `Tasks: Run Test Task` | ||
| - Run `colcon test --event-handlers=console_direct+` | ||
| The runs the tests and pipes the output to the terminal. To test `pi_main` make sure to type your password into the terminal after doing either option.. |
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.
grammar
Added notes to solve rendering issues and long boot times in UTM
for more information, see https://pre-commit.ci
ma4848
left a comment
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.
Looks good!
Pull Request
What type of PR is this? (check all applicable)
Description
Update the main README so that it's organization makes more sense when setting up. Also minor updates to wording in various sections.
Related Tickets & Documents