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

disable some tests for connext #145

Merged
merged 1 commit into from
Aug 29, 2019
Merged

disable some tests for connext #145

merged 1 commit into from
Aug 29, 2019

Conversation

Karsten1987
Copy link
Collaborator

CI (only Connext):

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

CI (only FastRTPS):

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Signed-off-by: Karsten Knese karsten@openrobotics.org

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
@Karsten1987 Karsten1987 self-assigned this Aug 7, 2019
@ivanpauno
Copy link
Member

aarch64 with only Connext is failing, because Connext isn't available in that platform.

@Karsten1987 Karsten1987 merged commit b7ed595 into master Aug 29, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix_connext_tests branch August 29, 2019 00:25
Karsten1987 added a commit that referenced this pull request Aug 29, 2019
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Karsten1987 added a commit that referenced this pull request Sep 9, 2019
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
thomas-moulard pushed a commit to thomas-moulard/rosbag2 that referenced this pull request Oct 22, 2019
* ros2GH-27 Implement discovery for new topics after startup

- this allows new topics to be discovered also after startup.
- as a consquence, ros2 bag record won't throw any exception anymore if the specified topics are not available at startup
- each time the recorder subscribes to a new topic, this will be logged to console

* ros2GH-27 Provide polling frequency as option

- make option available in RecordOptions
- set to 100ms for now

* ros2GH-27 Stop topic polling if subscription setup is complete

* ros2GH-27 Minor refactoring for readability

* ros2GH-27 Fix build after rebase

* ros2GH-145 add no-discovery option to ros2 bag record

* ros2GH-27 Refactor subscribing mechanism

- use set for subscribed topics instead of vector
- perform the asynchronous call only when discovery is needed

* ros2GH-27 Expose polling-interval as cli option

* ros2GH-27 Minor refactoring

* ros2GH-27 Refactor recorder

- naming of methods
- emphasize similarity between first subscription and discovery loop

* small touch ups

* remove launch from function name
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