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

Use ffmpeg connections #141

Merged
merged 3 commits into from
Mar 21, 2024
Merged

Use ffmpeg connections #141

merged 3 commits into from
Mar 21, 2024

Conversation

S-Dafarra
Copy link
Collaborator

Since robotology/yarp#3086 has been merged, we can also merge these configuration files

@S-Dafarra S-Dafarra merged commit 6d25595 into master Mar 21, 2024
@S-Dafarra S-Dafarra deleted the use_ffmpeg_connections branch March 21, 2024 08:45
@traversaro
Copy link
Member

Since robotology/yarp#3086 has been merged, we can also merge these configuration files

Should we document the dependency on YARP >= 3.10 on the readme? As it is a runtime dependency, you only realize that something is not working once you actually try to make the connection, and also in that case I guess the error will not be super clear.

@S-Dafarra
Copy link
Collaborator Author

Since robotology/yarp#3086 has been merged, we can also merge these configuration files

Should we document the dependency on YARP >= 3.10 on the readme? As it is a runtime dependency, you only realize that something is not working once you actually try to make the connection, and also in that case I guess the error will not be super clear.

Good point. I have opened #142 as a remainder. The idea is to do this modification once yarp 3.10 is released. At the moment, the set of users affected by this modification should be limited, hence waiting should not be a big issue.

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.

3 participants