-
Notifications
You must be signed in to change notification settings - Fork 29
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
Problem in example 1 #92
Comments
As far as I can see, the code reads
if (run. load ()) {
dataforAudioService (theRadio,
programName. c_str (), &ad, 0);
if (!ad. defined) {
std::cerr << "sorry we cannot handle service " <<
programName << "\n";
run. store (false);
}
}
if (run. load ()) {
dabReset_msc (theRadio);
set_audioChannel (theRadio, &ad);
}
While the code guarded by the second condition could have been merged with
the code guarded byh the first
condition, it is clear that if (a) "run" is set to true, and the service is
defined, the function "set_audioChannel"
is called.
So I am a little cofused by your statment, I just ran a test and it runs
fine, with sound
Op zo 14 mei 2023 om 22:04 schreef 1337Misom ***@***.***>:
… Hi,
I just wanted to mention you forgot to call
set_audioChannel(theRadio,&ad); after dataforAudioService() in examples 1
and 2. Without it doesn't start the decoding process (at least with an
rtlsdr).
—
Reply to this email directly, view it on GitHub
<#92>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACCPHQHYTU73OFE42RCZKGDXGE3FNANCNFSM6AAAAAAYBLK5EY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
--
Jan van Katwijk
|
The code you send would work ,but the one currently in the git doesn't include the if statement or any other form of calling set_audioChannel. It is correctly implemented in example 3. |
Then ther eis some miscommunication, this is the code - just extracted -
from example 2 ikn the repository
What code do you have, and whgere/how did you get/extract it
if (run. load ()) {
dataforAudioService (theRadio,
programName. c_str (), &ad, 0);
if (!ad. defined) {
std::cerr << "sorry we cannot handle service " <<
programName << "\n";
run. store (false);
}
}
if (run. load ()) {
dabReset_msc (theRadio);
set_audioChannel (theRadio, &ad);
}
while (run. load () && (theDuration != 0)) {
if (theDuration > 0)
theDuration --;
sleep (1);
}
Op ma 15 mei 2023 om 10:59 schreef 1337Misom ***@***.***>:
… The code you send would work ,but the one currently in the git doesn't
include the if statement or any other form of set_audioChannel
—
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACCPHQBKL7KH2MSV2EKNBN3XGHV73ANCNFSM6AAAAAAYBLK5EY>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Jan van Katwijk
|
Okay then it only seems to be a problem in example 1 |
You were right aaout example-1, it is adapted
In my defence I can say that the examples are - as mentioned in different
places -
* E X A M P L E P R O G R A M
* This program might (or might not) be used to mould the interface to
* your wishes. Do not take it as a definitive and "ready" program
* for the DAB-library
anyway, thanlks for the tip and good luck
Op ma 15 mei 2023 om 11:31 schreef 1337Misom ***@***.***>:
… Okay then it only seems to be a problem in example 2
—
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACCPHQHZ6UDCBYYPRYTT3BDXGHZXBANCNFSM6AAAAAAYBLK5EY>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
Jan van Katwijk
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I just wanted to mention you forgot to call
set_audioChannel(theRadio,&ad);
afterdataforAudioService()
in examples 1 and 2. Without it doesn't start the decoding process (at least with an rtlsdr).The text was updated successfully, but these errors were encountered: