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

you need .toString for this to work. #88

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

spanner3003
Copy link

Hope this helps

@RogerHardiman
Copy link
Collaborator

H
Thanks for the Pull Request.
I just had one quick question before I merge it.

What problem/error message were you seeing without the fix?

I just wanted to ask as the GStreamer code was written by someone else and they had it all working without .toString so wondered if this is a bug that shows up in newer versions of Node or newer JS runtimes or newer Typescript versions.

Thanks
Roger

@spanner3003
Copy link
Author

spanner3003 commented Apr 29, 2020

When I would start rpos.js after start live 555 streamer with debug on it was not parsing the -d the the device, I am using which is the usb camera so it was start gststreamer with picam settings.

@spanner3003 spanner3003 closed this May 3, 2020
@spanner3003
Copy link
Author

Oops you have not merged it.
Sorry.
Thank you for your work

@spanner3003 spanner3003 reopened this May 3, 2020
@RogerHardiman
Copy link
Collaborator

Hi.
Not had time to merge it yet.

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