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

Workspace Extension Functionality #36

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

vkmb
Copy link
Contributor

@vkmb vkmb commented Sep 27, 2022

By default, Zethus uses the ROS_PACKAGE_PATH set before Jupyter Hub launches.

To use a new ROS workspace inside of Jupyter Lab, the existing procedure is,

  1. Stop the jupyter lab instance.
  2. Source the new workspace.
  3. Start the jupyter lab instance.

This workflow is doable in a local instance. The problem arises with the Jupyter Hub instance launched from a Dockerfile.

In the Jupyter Hub instance with the current Jupyterlab-ros server plugin, it's impossible to add the new workspaces and fetch new robot models making it cumbersome to visualise robot models.

The updated plugin tries to solve this problem by enabling the user to add and delete workspaces as required for the task without restarting the Jupyter lab, copying the packages to the base path, modifying the Dockerfile and restarting Jupyter Hub instance.
To test the plugin

docker run --rm -p 8888:8888 kumkum/jupyterhub:jupyterlab-ros-stable

- Fixed recursion error for workspaces
- Fixed workspace description typo in settings
- Fixed viz-robotmodel.png not found
Workspace Extension Functionality
Copy link
Collaborator

@IsabelParedes IsabelParedes left a comment

Choose a reason for hiding this comment

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

Hi @vkmb ,

thanks for your PR! Looks great, just needs some minor changes 😁

js/schema/settings.json Outdated Show resolved Hide resolved
jupyter_ros_server/api/pkgs.py Outdated Show resolved Hide resolved
jupyter_ros_server/static/settings.json Outdated Show resolved Hide resolved
- Restored saved settings to default state
- Fixed typo in settings schema
- Removed unused import in Pkgs Module
@vkmb
Copy link
Contributor Author

vkmb commented Sep 28, 2022

Thanks a lot for the suggestions 👍 . I had fixed them in this ff46dff commit.

@hbcarlos
Copy link
Member

This is awesome. Thanks for working on this PR @vkmb!

Copy link
Collaborator

@IsabelParedes IsabelParedes left a comment

Choose a reason for hiding this comment

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

Excellent!

@IsabelParedes IsabelParedes added bug Something isn't working enhancement New feature or request labels Sep 29, 2022
@IsabelParedes IsabelParedes merged commit e9aa5e4 into RoboStack:master Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants