huxuan / iptvtools

A set of scripts that help to better the IPTV experience.
https://iptvtools.readthedocs.io/
MIT License
182 stars 50 forks source link

The script sometimes fails #23

Open werrent opened 2 years ago

werrent commented 2 years ago

Hi, I just installed iptv-filter today. I ran into a little problem when I tried to use it

I am not requesting assistance with this. I am only reporting this for your information

I use Ubuntu Linux 18.04, with python3 3.6.9 installed

I ran iptv-filter against some test data that I have, and it processed each file successfully. When it works, it seems to work nicely. I have been trying to write something similar. Your script is much faster and more feature-rich than anything that I have written

After testing your script, I downloaded the latest m3u file from https://satupdates.com/canada-iptv-m3u-playlist

I have gzipped the file and uploaded it. The link is on the next line

canada-270522.m3u.gz

When I try to run iptv-filter with this file, I see this:

iptv-filter -i canada-270522.m3u -o m27e.m3u Traceback (most recent call last): File "/home/werrent/.local/bin/iptv-filter", line 8, in sys.exit(main()) File "/home/werrent/.local/lib/python3.6/site-packages/iptvtools/iptv_filter.py", line 73, in main playlist.parse() File "/home/werrent/.local/lib/python3.6/site-packages/iptvtools/models.py", line 68, in parse self._parse(self.args.inputs) File "/home/werrent/.local/lib/python3.6/site-packages/iptvtools/models.py", line 137, in _parse if current_id not in self.id_url: UnboundLocalError: local variable 'current_id' referenced before assignment

huxuan commented 2 years ago

Hi @werrent, thanks for the reporting. It is actually a bug for iptvtools. iptvtools is not well tested and it expects to have a valid first line for the m3u. I think you have already find some workarounds. I will keep this issue open until the corresponding problem is fixed. But there is no promise on the ETA. Just let me know if you have any further questions.

werrent commented 2 years ago

Hi Xuan,

thanks for the message. that sounds fine to me, no worries

Randy

On Sat, May 28, 2022 at 6:06 AM Xuan (Sean) Hu @.***> wrote:

Hi @werrent https://github.com/werrent, thanks for the reporting. It is actually a bug for iptvtools. iptvtools is not well tested and it expects to have a valid first line for the m3u. I think you have already find some workarounds. I will keep this issue open until the corresponding problem is fixed. But there is no promise on the ETA. Just let me know if you have any further questions.

— Reply to this email directly, view it on GitHub https://github.com/huxuan/iptvtools/issues/23#issuecomment-1140227109, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGB2NADOIMNBEXRH226VGULVMHVZNANCNFSM5XFVNK7Q . You are receiving this because you were mentioned.Message ID: @.***>