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

Release Candidate 1 #1

Open
wants to merge 53 commits into
base: rc1
Choose a base branch
from
Open

Release Candidate 1 #1

wants to merge 53 commits into from

Conversation

Joshpolansky
Copy link
Member

What:

This is the first possible release for the beckhoff bridge. It supports setting up a connection and has a simple API to add variables to read.

Why:

The bridge simplifies the process of connecting to a Beckhoff PLC with Isaac sim

Notes:

Readme and other assets should be updated before official relesase

@@ -0,0 +1,65 @@
import pyads
Copy link
Member

Choose a reason for hiding this comment

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

@neumannjim do you have any thoughts on what license would be appropriate?

Copy link
Member

Choose a reason for hiding this comment

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

One of the other sample files from NVIDIA already had a license on it

Copy link
Member

Choose a reason for hiding this comment

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

MIT! I'll see if @agrayzel can help give this the Open Loupe treatment.

Joshpolansky and others added 22 commits April 25, 2024 11:18
This is now a general omni.kit extension
This is now reflected in the status messages displayed to the user
- Add some keywords
- Make it public so that it is auto added to sys.path
Co-authored-by: Shane Reetz <[email protected]>
- Evidently extensions are supposed to be called <company_name>/<category>/<app_name>
Prepare for official release
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.

5 participants