8000 docs: rai 2.0 update by maciejmajek · Pull Request #554 · RobotecAI/rai · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

docs: rai 2.0 update #554

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 35 commits into from
May 8, 2025
Merged

docs: rai 2.0 update #554

merged 35 commits into from
May 8, 2025

Conversation

maciejmajek
Copy link
Member
@maciejmajek maciejmajek commented Apr 30, 2025

Purpose

Finalizing RAI 2.0 with revamped documentation.

Proposed Changes

Docs

Issues

Closes 171

Testing

Eyes + python interpreter for code snippets: https://RobotecAI.github.io/rai/

local setup:

poetry install --with docs
mkdocs serve

@maciejmajek
Copy link
Member Author

@coderabbitai full review

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as outdated.

@maciejmajek maciejmajek force-pushed the docs/rai-2.0 branch 2 times, most recently from 85486a9 to 82ddcfe Compare May 3, 2025 19:00
@Cytrus14 Cytrus14 self-requested a review May 5, 2025 11:56
Copy link
Contributor
@Cytrus14 Cytrus14 left a comment

Choose a reason for hiding this comment

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

The documentation requires mostly typo fixes. A small number of other potential problems has also been identified and specified in appropriate comments.

Run the following command to build the whoami:

```bash
python src/rai_whoami/rai_whoami/build_whoami.py panda/ --build-vector-db
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: Running this command while using small models (e.g., llama3.1:8b) often leads (approximately 50% of the time) to various exceptions in the binascii.a2b_base64(s) method. Example:

Traceback (most recent call last):
  File "/home/<user>/Work/Robotec/rai/src/rai_whoami/rai_whoami/build_whoami.py", line 56, in <module>
    build_whoami(args)
  File "/home/<user>/Work/Robotec/rai/src/rai_whoami/rai_whoami/build_whoami.py", line 43, in build_whoami
    info.to_directory(args.output_dir)
  File "/home/<user>/Work/Robotec/rai/src/rai_whoami/rai_whoami/models/models.py", line 180, in to_directory
    f.write(base64.b64decode(image))
  File "/usr/lib/python3.10/base64.py", line 87, in b64decode
    return binascii.a2b_base64(s)
binascii.Error: Incorrect padding

Copy link
Member Author

Choose a reason for hiding this comment

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

Please create an issue for the rai_whoami package

@jmatejcz
Copy link
Contributor
jmatejcz commented May 7, 2025

the rosbot demo is not working
image
fix to https://github.com/RobotecAI/rai/blob/docs/rai-2.0/examples/rosbot-xl-demo.py needed


```bash
ros2 topic echo /to_human rai_interfaces/msg/HRIMessage
```
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 link manipulation demo in this walkthrough, so that it's possible to have something practically running at the end of this walkthrough. Without real panda robot or simulation, it'll be hard to see the effect of this tutorial.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's a tutorial, not a demo. It won't be easy to integrate the custom code with the demo. The only common thing is the generated whoami, but we will have the whoami already prepared for the manipulation demo.

Copy link
Member

Choose a reason for hiding this comment

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

It won't be easy to integrate the custom code with the demo.

One idea might be to just do a walkthrough using the manipulation demo and as an example reimplement for the sake of the tutorial one of already implemented tools. For example GetROS2Image tool is fairly simple. Then the tutorial can be interactive.

but we will have the whoami already prepared for the manipulation demo.

We might we the same mechanism at it was in the turtlebot demo. You can generate your own whoami or download a zip with pregenerated files.

Copy link
Member Author
@maciejmajek maciejmajek May 7, 2025

Choose a reason for hiding this comment

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

One idea might be to just do a walkthrough using the manipulation demo and as an example reimplement for the sake of the tutorial one of already implemented tools. For example GetROS2Image tool is fairly simple. Then the tutorial can be interactive.

Sure, but this is a walkthrough through rai core abstractions for new use-cases. If you feel like this is important please do a PR with new tutorial on that topic.

We might we the same mechanism at it was in the turtlebot demo. You can generate your own whoami or download a zip with pregenerated files.

A single, fine-tuned json file will be provided in the repository (for the demos).

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 merge this file with index.md, because some info is repeated. Or (less preferable) make mutual sections separate files.

Copy link
Member Author

Choose a reason for hiding this comment

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

The idea was to give an extended view of what rai is as well as answer common questions. The content will of course repeat, as the overview already provides some context.

Copy link
Member
@boczekbartek boczekbartek May 7, 2025

Choose a reason for hiding this comment

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

I understand and I think that it's a very good idea. I just think that structure of the what_is_rai is nice and should also be in the main page. I mean sections:

  • Quick Start
    ...
  • How RAI Works
  • Community & Support
  • Learn More
    If you think they should be in the what_is_rai page, it's fine to just copy-paste.

@maciejmajek
Copy link
Member Author
maciejmajek commented May 8, 2025

Moving issues to #461 (comment)

@boczekbartek boczekbartek self-requested a review May 8, 2025 10:01
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.

Approving. Rest of the comments will be resolved under this issue: #461 (comment)

@maciejmajek maciejmajek merged commit 4e22c74 into development May 8, 2025
5 checks passed
@maciejmajek maciejmajek deleted the docs/rai-2.0 branch May 8, 2025 10:12
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.

RAI 2.0 Documentation update Migrate documentation to GitHub pages
4 participants
0