音色番号#79(Whistle)のノイズと音程の修正

Issue #424 resolved
権兵衛 七篠 created an issue

七篠です。
先日のCWHSチャットで出た話題をもとに提案をいたします。
チャットでの新作シナリオのテスト中、
Pyエンジンで一部BGMにノイズがのる問題が検証され、
検証の結果、標準添付のサウンドフォント「TimGM6mb」に
表題のとおり問題があることが確認されました。

この件につきまして、チャットご同席のみしかさんが
ノイズ修正と音程の狂いを調律しましたファイルをおつくりなられましたので、
これを添付いたします。
※なお、みしか さんへのコンタクトが必要な場合は
Twitterの@misicawirthまたはサイトのメールフォームまで
ご連絡いただければ、とのことです。

Comments (14)

  1. 権兵衛 七篠 reporter

    なお問題の出たMIDIファイルの詳細は
    Ramine さんの「intermezzo -空の魚-」です。
    再生時間7-8秒目から入ってきます1番パートの音色番号79番(Whistle)で
    ノイズが乗ることを確認しています。

    個人的には、この修正サウンドフォントを
    適用することが望ましいと考えていますが、
    修正したサウンドフォントの利権面での扱いを聞き忘れていたので(汗
    万一問題あると判断なさられた場合は、
    このissueを「修正しない」ないしは「無効」あたりで
    始末していただいてもかまいません。

  2. 暗黒 騎士

    ライセンス面で不明瞭なものを取り込むわけにはいかないでしょうし、 みしか氏はフォローさせて頂いてるので確認してみます。(前回と同じかと思いますが念のため)

  3. 暗黒 騎士

    ライセンスの件は著作権が発生していないためPDのままというご見解でした。

    音源の差し替えなどは行っておらず、読み出し位置の修正と音高の修正のみなので当然パブリックドメインのままになるとおもいます!(当方は一切の権利を有しません)

    貢献者リストへの記載もOKとのことです。

  4. 権兵衛 七篠 reporter

    お手数おかけしスミマセン(汗
    前回のサウンドフォント関連issue(#382)は
    何度も確認してきたのに詰めの甘さで恐縮しきりです........

  5. 暗黒 騎士

    いえいえ、前回のように確認が何度も入るのはどうかと思ったのでお節介を焼かせて頂きました。 自分は1.30の時点で我慢ならず差し替えた口ですが、 デフォルトを使うというのは当たり前で、それでノイズが入るのはきついと思いますので、この提案は有意義に思います。

  6. k4nagatsuki repo owner

    問題の発見・修正・橋渡しとそのフォロー、ありがとうございます。

    サウンドフォントの構造の知識がさっぱり無いので自信を持っては言えませんが、issue #382とは別口の問題ですね。マージするのは問題ありませんし、データをご提供いただいたので、もちろん貢献者リストに掲載させていただきます。

    pull request #1617のレビュアーに@GhonbeNANASHINOさんを設定させていただきました。お手数ですがまた橋渡ししていただき、内容に問題が無いか確認して頂けないでしょうか。

    私は音周りは本当に弱いので、この修正を取り込む事で新たな問題が起きていたとしても気づく事ができません。もし何か発見されたら方がいらっしゃいましたら、ご指摘ください。おそらく実際には問題自体無いと思います。

  7. 暗黒 騎士

    すいません、また差し出がましいことを言いますが、このサイトの仕組みについて習熟されていない方にレビューさせるのは少々酷なのではないかと…。記述については一度マージしてからzipの方で確認して頂く方がよいかと思います。

    問題の検証をチャットの方複数人で行って頂いた流れで七篠さんが課題を立てられたという経緯のようですので、すでにファイルを入れ替えてのレビューは済んでいると思います。(自分も該当曲でノイズの発生と添付ファイルでそれが修正されることを確認しています)

  8. k4nagatsuki repo owner

    ファイル自体はたぶん問題ないのですが、テキストの記述の方で問題があるかもしれませんので、できれば確認していただきたいのです。

    そしてテキストはpull request #1617に差分が表示されるはず(されてますよね?)なので、それを見て確認していただければ大丈夫ではないかと思います。

    もちろん、大丈夫でなければご質問いただければ対応させていただきます。


    というかなぜかTimGM6mb.sf2.txtがSJISになっているせいで文字化けしていましたね。UTF-8に直しました。

  9. k4nagatsuki repo owner

    ところで、今回、ライセンスや貢献者リストへの掲載についてすでに確認が取れているにも関わらず@GhonbeNANASHINOさんを通してみしかさんに最終的な文面のチェックをしていただくというまだるっこしく迂遠な事をしているわけですが、報告者に対して余計な手間を掛けさせてしまうような、こうした姿勢はいかがなものかという考えがある事も理解しているつもりです。

    はっきりした基準を設ける事は困難ですが、今回の文面程度の事なら、とっととマージしてしまって問題があったら後からご指摘ください、というやり方の方がよいのかもしれません。

    この辺りは私も延々と試行錯誤が終わらない状態ですので、どうかご勘弁ください。

  10. 権兵衛 七篠 reporter

    えーと、れ、れびゅわー?
    あーっと、つまり、修正サウンドフォントについて、
    今度のチャットで、「提案の結果採用の運びとなりまして、
    それに伴いまして添付の文章がこんな格好で更新の予定です」
    ....といった報告を上げて、了承得られましたらば、
    このpull request #1617のページ右上にある
    「承認」のボタンを私めが押すことで、
    開発中のCWPyとのファイナルフュージョン....もとい、
    マージが行われる、みたいな調子で宜しいのでしょうか.....?


    タレコミ人の負担に関しては、
    うまく文章をまとめるのに失敗しましたので結論だけですが
    とりあえず、うちはかまいませんよ~、と暫定的にコメントしてみます。

  11. k4nagatsuki repo owner

    はい、その通りです。レビューの結果を見て最終的にマージ作業を行うのはマージ権限を持つ私になりますので、私に結果が伝わればどんな方法でもOKです(承認ボタンを押すのが一番楽だと思いますが)。

    お手数をおかけして本当に申し訳ないのですが、どうぞよろしくお願いいたします。

  12. 権兵衛 七篠 reporter

    七篠です。先ほどチャットで了承いただけましたので
    小声で叫んで「承認」してみました。あとは実際のマージ結果で
    問題がないかどうかを確かめたらクローズ、みたいな感じでしょうか。
    ノイズの原因とその対策についても話をうかがうことが出来ましたので、
    (クローズに間に合えば(汗))追加で記事をレスしたいと思います。

  13. k4nagatsuki repo owner

    ありがとうございます。マージしました。

    お手間を取らせて申し訳ありませんでした。上の方にも書きましたが、今後は些細と思える事は事後承諾をいただくつもりでどんどん進めていこうと思っています。

    その時々でうっかり方針に反した行動を取ってしまう事もあるはずなので、「こいつの行動なんか事変だぞ」と思ったらまた遠慮なくご指摘ください。

    クローズは七篠さんの任意のタイミングでしていただいてOKです。

  14. Log in to comment