Uploaded image for project: 'Picard'
  1. Picard
  2. PICARD-1128

Support Microsoft WAVE format (RIFF/WAVE) tagging with ID3

XMLWordPrintable

    • Icon: New Feature New Feature
    • Resolution: Fixed
    • Icon: Normal Normal
    • 2.4
    • None
    • Tags & Metadata
    • None

      Implement support for Microsoft WAV audio format and corresponding tagging.

      At the moment Picard has weird behaviour working with WAVE files:

      • It creates an ID3v2 header in the WAVE file, but is saving tags under the common names instead of the ID3v2 mapped names
      • It is not able to read back these tags.

      First step is probably to implement this in the mutagen Python module. (see issue #207).

          [PICARD-1128] Support Microsoft WAVE format (RIFF/WAVE) tagging with ID3

          Philipp Wolfer added a comment -

          I reopen this for now. We still need to include a mutagen version for the Windows / macOS builds which actually supports this. Let's see how things go on the mutagen side. If there is a release we can just update. If not we could consider shipping with a development version of mutagen or delay this feature.

          Philipp Wolfer added a comment - I reopen this for now. We still need to include a mutagen version for the Windows / macOS builds which actually supports this. Let's see how things go on the mutagen side. If there is a release we can just update. If not we could consider shipping with a development version of mutagen or delay this feature.

          GitHub Bot added a comment -

          See code changes in pull request #1532 submitted by phw.

          GitHub Bot added a comment - See code changes in pull request #1532 submitted by phw .

          Philipp Wolfer added a comment -

          @Moonbase: Yes, using ID3 in a special chunk is pretty common and actually widely supported. Picard itself does not yet support this, hence this ticket As the proposed patch for Mutagen shows the chunk ID would be 'id3': https://github.com/quodlibet/mutagen/pull/321/files#diff-6adf812c7b1bbeed9c45bc7dbbbd86cfR160

          Philipp Wolfer added a comment - @Moonbase: Yes, using ID3 in a special chunk is pretty common and actually widely supported. Picard itself does not yet support this, hence this ticket As the proposed patch for Mutagen shows the chunk ID would be 'id3': https://github.com/quodlibet/mutagen/pull/321/files#diff-6adf812c7b1bbeed9c45bc7dbbbd86cfR160

          Moonbase added a comment - - edited

          I worked with Erik on libsndfile to include support for BWF v2 (Broadcast Wave Format, including a "bext" chunk in RIFF type files like WAV). It might be nice to show the loudness values in Picard, but I’m not sure.

          Do "normal users" (and Picard) tag WAV files using ID3v2? If so, using the "ID3 ", "ID32" or "id32" chunk?

          Moonbase added a comment - - edited I worked with Erik on libsndfile to include support for BWF v2 (Broadcast Wave Format, including a "bext" chunk in RIFF type files like WAV). It might be nice to show the loudness values in Picard, but I’m not sure. Do "normal users" (and Picard) tag WAV files using ID3v2? If so, using the "ID3 ", "ID32" or "id32" chunk?

          Philipp Wolfer added a comment -

          I have a first implementation based on Borewit's pull request at https://github.com/phw/picard/tree/PICARD-1128-wave-id3-tagging

          A small correction to the ticket description: Picard never created ID3v2 headers in WAV files. The `_save` implementation for WAVFile always was an empty function not writing anything to disc

          Philipp Wolfer added a comment - I have a first implementation based on Borewit's pull request at https://github.com/phw/picard/tree/PICARD-1128-wave-id3-tagging A small correction to the ticket description: Picard never created ID3v2 headers in WAV files. The `_save` implementation for WAVFile always was an empty function not writing anything to disc

          Borewit added a comment -

          Created a pull request on mutagen (Picard dependency) to propose adding RIFF/WAVE functionality. Currently supporting ID3v2 to store metadata.

          Borewit added a comment - Created a  pull request on mutagen  (Picard dependency) to propose adding RIFF/WAVE functionality. Currently supporting ID3v2 to store metadata.

            outsidecontext Philipp Wolfer
            Borewit Borewit
            Votes:
            2 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Version Package
                2.4