【要望】MirakurunからBonDriver_Mirakurunへの送信をチャンネルごとではなくサービスごとに（ネットワーク帯域削減のため） about mirakurun

[kanreisa]

大変良いご要望だと思います。
お手数ですが下記リポジトリで再度Issueの作成をお願いいたします。
https://github.com/Chinachu/BonDriver_Mirakurun

[kanreisa]

大変失礼いたしました。よく見たところ既に作成されていましたので再openしておきました。
Chinachu/BonDriver_Mirakurun#3

ちなみに Mirakurun 側のAPIでは既にサービス選択できるようになっています。
APIドキュメント: http:// Mirakurun :40772/swagger-ui/?url=/api/docs#!/services/getServiceStream

[akimasa]

https://github.com/akimasa/Mirakurun/tree/delpids-dev
BonDriver_Mirakurunで不要なPIDを受信しないためにMirakurunに改変を加えてみました。
今のMirakurunのAPIでは指定されたPIDを送信させない機能はないですよね…？

改変を加えたMirakurunでは今のところ、指定されたサービスのストリームでしかPIDの削除が機能しない試作の状態になっています。
よろしければプルリクエストを送って、コードを見ていただき、修正点がありましたら指摘していただけたらと思うのですが、いかがいたしましょう。
あるいは、自分で書いた方が早いとお考えなら、その方が良いかもしれません。

[kanreisa]

nullパケットについては既に削除しているため、その他PIDを削除するメリットがいまいち分かっていません。
ちなみに、APIリクエストによって渡された文字列をそのままTSFilterで処理してはいけません。

[akimasa]

そうですね。
確かに自分も作ってみて、指定されたPID（PSI/SI）を削除するメリットがよくわかりませんでした。
ビットレートが目に見えて減るわけではないみたいですし、苦労して削除する割に合わない気がします。
node-aribts v1のcheck_drop.jsで眺めて見た限りでは、PSI/SIは全体の1%程度しか占めないみたいでした。
epgdatacapbonさんは、どうお考えなのか気になります。

//コードへのご指摘ありがとうございます。毎回へぼいコードで、すみません。

[tkmsst]

akimasaさんコードありがとうございます。確かに費用対効果についてはあまり考えてませんでした。コードがかなり煩雑になる場合は別ですが、複数のtsを同時に転送することもあるので単純にネットワーク帯域が少しでも減らせればそれに越したことはないという考えでした。

[akimasa]

TSFilterの_processPacketで全てのパケットに対して削除するPIDかどうか調べて削除する処理を追加していました。これを追加すると、CPU使用率が上がる懸念があります。1%の削減のために、CPUを使うのはどうかなと思っていました。
しかし、よく考えたら元々提供するPIDの配列にあるかどうかチェックしているみたいのので、_providePidsからいらないPIDを消せば良かったみたいです。
これならおそらく機能を追加しても負荷が上がらないのでは無いかと思います。

TSFilterに文字列を直接渡していた件は直しておきました。
他にも問題点があるかと思いますが、指摘していただければ可能な限り直します。
それ以前に、指定されたPIDを削除する機能を採用するかどうかという問題がありますが…。

[kanreisa]

引き続き下記のスレで議論できます
Chinachu/BonDriver_Mirakurun#3

EPG情報を削るオプションについては今後も検討してまいります。