-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add PipeWire support #16
Conversation
Words cannot describe how cursed some of the code I just wrote are. |
Current issues:
|
lawl/* repositories seem to have been deleted. This commit fixes that.
aa5602c
to
1375439
Compare
1375439
to
39a4765
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore this review please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small changes
dst[n%framesz][n/framesz] = reader.next() | ||
} | ||
mu.Unlock() | ||
sampleDuration := time.Duration( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currently the way the buffer is read means this will sometimes be too short. can we double this (or just 1.5x) for now and fix it properly later with some refactoring?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I made it so that it's 2x until the read expires once, then it's 1x for all future deadlines until it's sending samples again.
39a4765
to
81e4318
Compare
Usage:
gr -b pipewire -d 'he400i-rtings-sink.eq'