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

福岡・大分の放送局でコメントが表示出来なかったので修正 #1

Closed
wants to merge 2 commits into from

Conversation

Till0196
Copy link

@Till0196 Till0196 commented Dec 20, 2020

KBCテレビ、RKB毎日放送、OBS大分放送は、元データ通り(最後に「1」不要)で動作しますが、それ以外の放送局は修正が必要でした。
テレビ西日本(フジテレビ系列)は修正を加えてもコメントが表示されませんでした。
※KBC九州朝日放送はKBCテレビに改名しています。

KBCテレビ、RKB毎日放送、OBS大分放送は、元データ通り(最後に「1」不要)で動作しますが、それ以外の放送局は修正が必要でした。
福岡と大分の放送局はこれで正常に動作することを確認しました。
※KBC九州朝日放送はKBCテレビに改名しています。
テレビ西日本の改名を反映していなかったので、修正しました。
@tsukumijima
Copy link
Owner

tsukumijima commented Dec 20, 2020

現行の TVRemotePlus では、放送局名と実況 ID の対照に ch_sid.tsv ではなく channel_table.json を利用しており、ch_sid.tsv は(今はもはや機能していない)実況勢いの取得処理にのみ利用されているはずです(レガシーコードが残っているだけ…)。

あと、新しいニコニコ実況の放送局名と実況 ID の対照は getNicoJikkyoID() で行われていますが、正規表現でチャンネル名の末尾に数字が入っていてもマッチするようにしていたはずなので、ちょっと謎です。
KBCテレビの改名に関してはそのまま追記して良いと思いますが、以前から .ch2 ファイルを変えていない(= ch2 内の表記が KBC 九州朝日放送のまま)環境があることを鑑みると、置き換えるのではなく新たに下に追記する形の方が良いかと思います。

こちらで確認したいので、.ch2 ファイルの当該箇所を貼っていただけないでしょうか。

@Till0196
Copy link
Author

少し前(旧ニコニコ実況時代)に修正して上手く行ってたので、特に報告してなかったのですが、リニューアル後は見ているとこが変わっていたのですね。
仕様変更に気づかず失礼しました。
また、新仕様でも地方局への対応を残して頂けて、ありがたいです。

リクエストは無視して、新たに修正(追記)して頂けると幸いです。

手元のch2の情報を貼っておきます。

福岡(※J:COM北九州エリア、民法放送は県内全域共通です)
KBCテレビ,0,0,1,1,56336,31874,31874,1
KBCテレビ2,0,0,1,1,56337,31874,31874,0
KBCテレビ,0,0,1,192,56720,31874,31874,0
NHKEテレ1北九州,0,1,2,1,56840,30849,30849,1
NHKEテレ2北九州,0,1,2,1,56841,30849,30849,0
NHKEテレ3北九州,0,1,2,1,56842,30849,30849,0
NHK携帯2,0,1,2,192,57224,30849,30849,0
NHK総合1・北九州,0,2,3,1,56832,30848,30848,1
NHK総合2・北九州,0,2,3,1,56833,30848,30848,0
NHK携帯G・北九州,0,2,3,192,57216,30848,30848,0
RKB毎日放送,0,3,4,1,56344,31875,31875,1
RKB毎日放送2,0,3,4,1,56345,31875,31875,0
RKB毎日放送携帯,0,3,4,192,56728,31875,31875,0
FBS福岡放送1,0,4,5,1,56352,31876,31876,1
FBS福岡放送2,0,4,5,1,56353,31876,31876,0
FBS福岡放送携帯,0,4,5,192,56736,31876,31876,0
TVQ九州放送1,0,5,7,1,56360,31877,31877,1
TVQ九州放送2,0,5,7,1,56361,31877,31877,0
TVQ九州放送携帯,0,5,7,192,56744,31877,31877,0
テレビ西日本1,0,6,8,1,56368,31878,31878,1
テレビ西日本2,0,6,8,1,56369,31878,31878,0
テレビ西日本,0,6,8,192,56752,31878,31878,0

大分(※ケーブルテレビ経由の受信です)
NHK総合1・大分,0,0,0,1,61440,31792,31792,1
NHK総合2・大分,0,0,0,1,61441,31792,31792,0
NHK携帯G・大分,0,0,0,192,61824,31792,31792,0
NHKEテレ1大分,0,1,2,1,61448,31793,31793,1
NHKEテレ2大分,0,1,2,1,61449,31793,31793,0
NHKEテレ3大分,0,1,2,1,61450,31793,31793,0
NHK携帯2,0,1,2,192,61832,31793,31793,0
OBS大分放送,0,2,3,1,61456,31794,31794,1
OBS大分放送2,0,2,3,1,61457,31794,31794,0
OBS大分放送携帯,0,2,3,192,61840,31794,31794,0
TOSテレビ大分1,0,3,4,1,61464,31795,31795,1
TOSテレビ大分2,0,3,4,1,61465,31795,31795,0
TOSテレビ大分3,0,3,4,1,61466,31795,31795,0
TOSテレビ大分携帯,0,3,4,192,61848,31795,31795,0
OAB大分朝日放送1,0,4,5,1,61472,31796,31796,1
OAB大分朝日放送2,0,4,5,1,61473,31796,31796,0
OAB大分朝日放送3,0,4,5,1,61474,31796,31796,0
OAB大分朝日 携帯,0,4,5,192,61856,31796,31796,0
FBS福岡放送1,0,5,5,1,56352,31876,31876,1
FBS福岡放送2,0,5,5,1,56353,31876,31876,0
FBS福岡放送携帯,0,5,5,192,56736,31876,31876,0
TVQ九州放送1,0,6,7,1,56360,31877,31877,1
TVQ九州放送2,0,6,7,1,56361,31877,31877,0
TVQ九州放送携帯,0,6,7,192,56744,31877,31877,0
テレビ西日本1,0,7,8,1,56368,31878,31878,1
テレビ西日本2,0,7,8,1,56369,31878,31878,0
テレビ西日本,0,7,8,192,56752,31878,31878,0

よろしくお願いします。

@tsukumijima
Copy link
Owner

ありがとうございます!時間のあるときに確認しておきます。

@tsukumijima
Copy link
Owner

682
683

Commit 703d6db で修正しました!
テレビ西日本は「TNCテレビ西日本」と定義されていたのが原因みたいです。
首都圏在住なのでとても助かります。ありがとうございました!

(余談ですが、随分前に大分に住んでいた事があり懐かしいなーと思いながら作業してました、あそこはケーブルに入らないと人権ないんですよね確か( KBC と RKB の区域外再放送が終了していたことをこれで知った))

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