Revert "Comments"

This reverts commit c39e054.

Author: NicolasHug

src/torchcodec/decoders/_blocks/_demuxer.py

@@ -261,6 +261,10 @@ def scan(self) -> StreamIndex:
         )
 
 
+# TODO_API_BREAKDOWN FEAT P1 no scan() here: StreamIndex is video-shaped
+# (keyframes, frame indices, fps), and audio has none of those notions. What an
+# audio scan should return - exact duration, total sample count - is part of the
+# StreamIndex redesign above.
 class AudioDemuxer(_BaseDemuxer):
     """Demux building block: opens a container and yields the compressed
     :class:`Packet`\\ s for one audio stream. Does no decoding.

test/test_decoders.py

@@ -5051,7 +5051,9 @@ def test_audio_demuxer_source_kinds(self, make_source):
         assert len(list(AudioDemuxer(make_source(NASA_AUDIO_MP3.path)))) == expected
 
     def test_audio_demuxer_seek(self):
-        # Seeking past the start leaves fewer packets to demux.
+        # Seeking past the start leaves fewer packets to demux. We don't assert
+        # anything about *which* ones: audio has no keyframe to land on, so
+        # where a seek lands is the demuxer's business, not something we pin.
         num_packets_from_start = len(list(AudioDemuxer(NASA_AUDIO_MP3.path)))
 
         demuxer = AudioDemuxer(NASA_AUDIO_MP3.path)
@@ -5060,6 +5062,10 @@ def test_audio_demuxer_seek(self):
 
         assert 0 < num_packets_after_seek < num_packets_from_start
 
+    def test_audio_demuxer_has_no_scan(self):
+        # See the TODO in _demuxer.py: StreamIndex is video-shaped.
+        assert not hasattr(AudioDemuxer(NASA_AUDIO_MP3.path), "scan")
+
     # ===== Audio decoding: RawAudioSamples =====
 
     @staticmethod