Skip to content

Conversation

@IsabelParedes
Copy link
Contributor

Public API Changes

None

Description

This PR simply fixes a few typos in the documentation.

Copy link
Collaborator

@wayneparrott wayneparrott left a comment

Choose a reason for hiding this comment

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

lgtm

@wayneparrott
Copy link
Collaborator

@ihuicatl thx for the type fixes. much appreciated.
OT - ros2wasm looks interesting. I've wondered about use of wasm in ros for awhile.

@IsabelParedes
Copy link
Contributor Author

Hi @wayneparrott, thanks for the review!
I was actually looking into possibly integrating rclnodejs into the ros2wasm project. So if you have any tips I'd be pretty interested :)

@minggangw
Copy link
Member

@ihuicatl thanks for fixing the typos!

@minggangw minggangw merged commit 369e765 into RobotWebTools:develop Apr 8, 2023
@minggangw
Copy link
Member

Hi @wayneparrott, thanks for the review! I was actually looking into possibly integrating rclnodejs into the ros2wasm project. So if you have any tips I'd be pretty interested :)

Never think about compiling rclnodejs to wasm, interesting! I'm not clear how you can deal with the addon written in C++, please update us if you have progress :) By the way, is there any motivation for you to leverage wasm?

@IsabelParedes IsabelParedes deleted the typos branch April 11, 2023 08:34
@IsabelParedes
Copy link
Contributor Author

Hi @minggangw! Currently the motivation is from an educational perspective. I worked on setting up a ROS environment with JupyterHub for a university course before, and having a ROS environment run entirely on the browser would be a step up from that.
Right now I have some publisher and subscriber nodes running on the browser but I'm looking to add more interaction which is where integrating rclnodejs would be really nice to have.
For the addon in C++, that should be compile-able with Emscripten which is how I've been compiling the rest of the ROS packages. I think I'll have to figure out how to detect my wasm modules as a valid ROS installation though.

@minggangw
Copy link
Member

@ihuicatl I see, that makes sense. Have you tried with roslibjs, which can support ROS2 on the browser, or Electron? Because both can save a lot of effort if you want to enable ROS2 on the browser.

@IsabelParedes
Copy link
Contributor Author

Hi @minggangw, I took a look at roslibjs before but wasn't sure if it supported ROS2 since I only saw one ros2 branch. But roslibjs seems to rely on a rosbridge connection to a native ROS installation, which I feel like I would run into the same problem that I'd have to figure out how to connect roslibjs to wasm modules without relying on a native ROS installation nor rosbridge.
I haven't used Electron before but that seems more for creating desktop applications which is not exactly what I'm trying to do. I'm simply trying to have ROS2 run on a website without requiring any installations by the user.

minggangw pushed a commit that referenced this pull request Apr 17, 2023
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.

3 participants