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

DOC-585: starting connext version-specific connector-py instructions #192

Open
wants to merge 2 commits into
base: support/connector/1.2.0
Choose a base branch
from

Conversation

rkorte
Copy link
Collaborator

@rkorte rkorte commented Nov 8, 2024


.. code:: bash

$ pip install rticonnextdds_connector

The above command installs the latest version of *Connext* by default. To
Copy link
Collaborator Author

@rkorte rkorte Nov 8, 2024

Choose a reason for hiding this comment

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

Obviously, these are not the right instructions. I do not know what the right instructions are. Elaine told me Connector-py needs the same changes that we made to Connector-js and to the Connext Python API regarding versions. So, @Alxe, please tell me what we should really say here (feel free to make changes in the branch if you prefer).

Copy link
Member

Choose a reason for hiding this comment

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

The pip documentation is accurate, but I'd prefer using rticonnextdds-connector (both are valid) so that it matches JS documentation.

@rkorte rkorte requested a review from Alxe November 8, 2024 20:01
@@ -7,21 +7,34 @@ Getting Started
Installing RTI Connector for Python
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

There are two ways to obtain *RTI Connector* for Python. You can install it with
*pip*:
There are two ways to obtain *RTI Connector* for Python: with
Copy link
Member

Choose a reason for hiding this comment

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

There's only one way. You can no longer clone the Repository.


.. code:: bash

$ pip install rticonnextdds_connector

The above command installs the latest version of *Connext* by default. To
Copy link
Member

Choose a reason for hiding this comment

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

The pip documentation is accurate, but I'd prefer using rticonnextdds-connector (both are valid) so that it matches JS documentation.

$ pip install rticonnextdds_connector==<version>

where ``<version>`` is any valid *Connext* version in the
`Release history <https://pypi.org/project/rti.connext/#history>`__
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`Release history <https://pypi.org/project/rti.connext/#history>`__
`Release history <https://pypi.org/project/rticonnextdds-connector/#history>`__

And then run your *Connector* applications:

.. code:: bash

$ python my_connector_app.py

You can also clone the repository and run the examples directly without installing
*Connector*:
**To clone the repository and run the examples directly without installing
Copy link
Member

Choose a reason for hiding this comment

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

Remove from here until Running the examples.

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.

2 participants