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

Python import statements for ambf_comm, ambf_world, etc. #239

Open
juseonghan opened this issue Nov 8, 2024 · 1 comment
Open

Python import statements for ambf_comm, ambf_world, etc. #239

juseonghan opened this issue Nov 8, 2024 · 1 comment

Comments

@juseonghan
Copy link

Hello - we are trying to create an AMBF wrapper to test and deploy RL algorithms, and we found your AmbfEnv() class that you specified in ambf_env.py. However, if I run env_test.py I get import errors. Upon closer inspection, it seems like import statements generally fail if it's not from ambf_client or ambf_msgs (e.g. import ambf_world, import ambf_comm all do not work). Is there a fix for this or are we doing something incorrectly from our side? We have ROS and AMBF sourced, and the client works fine otherwise. Thanks!

adnanmunawar added a commit that referenced this issue Nov 12, 2024
Move ambf_env to tests folder and rename to ambf_gym_env
@adnanmunawar
Copy link
Member

Hello, it has been a while since we used this functionality, please try updating your repo to include the commits above and check if this fixes the issue.

Broadly, you may want to check out this repo for our current integration with gym https://github.com/surgical-robotics-ai/SurgicAI

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

No branches or pull requests

2 participants