Skip to content

docs: update docs to reflect rai on pypi and interfaces on apt #612

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

Merged
merged 15 commits into from
Jun 3, 2025

Conversation

maciejmajek
Copy link
Member

Purpose

To provide instructions for anyone wanting to install rai with pip

Proposed Changes

Documentation changes

Issues

#607

Testing

  • How was it tested, what were the results?

@maciejmajek maciejmajek changed the title docs: update rai core and whoami readmes docs: update docs to reflect rai on pypi and interfaces on apt May 29, 2025
1. Install rai whoami:

```bash
pip install rai-whoami
Copy link
Member

Choose a reason for hiding this comment

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

Same considerations for ubuntu 24.04 as in rai-core.

Copy link
Member

Choose a reason for hiding this comment

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

@maciejmajek I think it would be beneficial to mention the tip about ros2 packages here as well


When installing RAI in a virtual environment (as opposed to the system interpreter), you need to expand the PYTHONPATH to include the virtual environment's site-packages directory.

Here's how to do it when RAI is installed via git clone: [setup_shell.sh](https://github.com/RobotecAI/rai/blob/245efa95cdb83a81294bc28da814962bff84be20/setup_shell.sh#L32)
Copy link
Member

Choose a reason for hiding this comment

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

This will only work if virtualenv is created with the same python version as ros2 native python.
I think it would be good to mention that as well. Python version used in ros2 can be different depending on ubuntu version (u22-python3.10, u24-python3.12). This can be checked using:

find /opt/ros/${ROS_DISTRO}/lib/ -type d -name "python3.*" | rev | cut -f1 -d '/' | rev

.
Alse, I tried installing rai in uv virtualenv and had such error, because uv used python3.11 by default. To make it work I had to use python3.10 explicitely:

cd workdir
uv venv --python 3.10
uv add rai-core

And I managed to run react.py example using:

cd workdir
source .venv/bin/activate
python react.py

Copy link
Member Author

@maciejmajek maciejmajek Jun 2, 2025

Choose a reason for hiding this comment

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

Fokin hell m8. A simplistic way of installing rai is getting super complex. Nevertheless, yeah, it should be mentioned

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@maciejmajek
Copy link
Member Author

maciejmajek commented Jun 3, 2025

A note should be added that it's advised to run the demos with poetry install
done

Copy link
Member

@boczekbartek boczekbartek left a comment

Choose a reason for hiding this comment

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

@maciejmajek Leaving some minor style comments. Otherwise LGTM!

1. Install rai whoami:

```bash
pip install rai-whoami
Copy link
Member

Choose a reason for hiding this comment

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

@maciejmajek I think it would be beneficial to mention the tip about ros2 packages here as well


## Installing RAI

??? tip "Virtual environment"
Copy link
Member

Choose a reason for hiding this comment

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

I would make it untoggled by default

Co-authored-by: Bartek Boczek <[email protected]>
@maciejmajek maciejmajek merged commit 3b2dcbc into development Jun 3, 2025
6 checks passed
@maciejmajek maciejmajek deleted the docs/pypi branch June 3, 2025 13:27
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.

2 participants