search

stefantalpalaru / w_scan2

w_scan fork (channel scan tool which generates ATSC, DVB-C, DVB-S/S2 and DVB-T/T2 channels.conf files)
GNU General Public License v2.0
80 stars 22 forks source link

delsys based on countries #6

Closed DF5EQ closed 5 years ago

DF5EQ commented 5 years ago

Now you can close the former PRs and may use this one.

stefantalpalaru commented 5 years ago

Changing delsysloop_max() to always scan for DVB-T2 was done on purpose? Looks like it's used outside terrestrial broadcasts, increasing scanning time for no reason.

DF5EQ commented 5 years ago

The scanning of both (DVB-T and DVB-T2) was introduced in w_scan-20170107. There the functions delsysloop_min/max were added in countries.c/.h and used in scan.c function initial_tune. See commit a8a8c07eec964e06e3b649315ea191f9c8ee60c5 "merge w_scan-20170107". So I didn't change the default behavior. I changed the delsysloop functions to be able to set the desired looping (only DVB-T, only DVB-T2, both) on a per country basis instead of a country list basis. This gives a more fine control. I set the looping to only DVB-T2 for DE and AT. CO was already set in version 20170107. So if somebody knows the current state of the DVB systems used in different countries he/she is invited to add them. As default we have two loops so we don't miss anything. I made the change, because it was time wasting for me to wait for the DVB-T loop in AT :)

Am 08.10.2018 um 19:13 schrieb Ștefan Talpalaru:

Changing delsysloop_max() to always scan for DVB-T2 was done on purpose? Looks like it's used outside terrestrial broadcasts, increasing scanning time for no reason.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stefantalpalaru/w_scan2/pull/6#issuecomment-427912022, or mute the thread https://github.com/notifications/unsubscribe-auth/Ap4Mbh_02WBcb5JsxyekehdBFZ1AiSHWks5ui4fTgaJpZM4XLiiC.

stefantalpalaru commented 5 years ago

Merged. Thanks.

DF5EQ commented 5 years ago

Thanks for taking my commits. Sorry for not answering in the last few days, but I was on a very busy business trip.

I think you have seen that I saved the former behavior and only changed the granularity for selecting the delsys loops.

Am 13.10.2018 um 00:52 schrieb Ștefan Talpalaru:

Merged #6 https://github.com/stefantalpalaru/w_scan2/pull/6 into master.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/stefantalpalaru/w_scan2/pull/6#event-1901844126, or mute the thread https://github.com/notifications/unsubscribe-auth/Ap4MbuzqUUMd9_M9X6yGC0FPQ7ptIk_hks5ukR1FgaJpZM4XLiiC.