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

[Suggestion] include staff/(premium=2) comments/command for download #1

Open
fireattack opened this issue Sep 21, 2024 · 1 comment

Comments

@fireattack
Copy link

(Sorry for writing in English, my Japanese is not good enough. A GPT-translated version is attached below.)

Thanks for the great lib.

Currently, it doesn't seem to download comments (shown as a banner in nico livestream) and various commands (such as /perm, /vote new ..., etc.) from staff members, compared to what NiconamaCommentViewer downloads. These data are crucial for fully re-implementing danmaku functionality locally.

So I was wondering if you can add options to include these comments for download as well.


(日本語があまり得意ではないので、英語で失礼します。以下にGPTで翻訳した日本語版も添付しています。)

素晴らしいライブラリをありがとうございます!

今のところ、ニコ生配信中に表示されるバナーコメントや、/perm/vote new ... などのスタッフコマンドがNiconamaCommentViewerと比べてダウンロードされていないようです。これらのデータは、ローカルで弾幕機能をちゃんと再現するために重要だと思います。

そこで、download にこれらのコメントも含めるオプションを追加できないか、検討していただけると嬉しいです。

@tsukumijima
Copy link
Owner

tsukumijima commented Sep 21, 2024

@fireattack

今のところ、ニコ生配信中に表示されるバナーコメントや、/perm、/vote new ... などのスタッフコマンドがNiconamaCommentViewerと比べてダウンロードされていないようです。これらのデータは、ローカルで弾幕機能をちゃんと再現するために重要だと思います。

このライブラリはニコニコ実況のコメントをダウンロードする目的で実装されたものです。ニコニコ実況では運営コマンドなどは必要とされません。
加えて、README にも記載されている通り、NDGR 新メッセージサーバーではいわゆる「運営コマンドコメント」の概念がなくなりました。
運営コマンドはスラッシュから始まる特殊なコメントではなく、Protobuf で構造化された状態で送信されます。

要はデータ構造が別物になり、また私は従来のメッセージサーバーで送られていた運営コメントの仕様に詳しくない(そして今となっては仕様を確認することもできない)ため、それらを「運営コマンドコメント」として従来のメッセージサーバー互換の XML に変換するのは困難です。
もちろんプルリクエストは歓迎しますが、最近多忙なこともあり、すぐに対応を入れる予定はありません。

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

No branches or pull requests

2 participants