Nudged version

Updated python-publish to latest version

.flake8

@@ -0,0 +1,5 @@
+[flake8]
+per-file-ignores =
+    pycmx/__init__.py: F401
+    tests/__init__.py: F401
+    

.github/workflows/bsky_test.yml

@@ -0,0 +1,25 @@
+name: Test Post to Bluesky
+
+on:
+  workflow_dispatch:
+    
+# permissions:
+#   contents: read
+#   id-token: write
+  
+jobs:
+  deploy:
+    runs-on: ubuntu-latest
+    environment:
+      name: release
+    steps:
+    - name: Send Bluesky Post
+      uses: myConsciousness/bluesky-post@v5
+      with:
+        text: |
+          This is a test post!
+        link-preview-url: ${{ github.server_url }}/${{ github.repository }}
+        identifier: ${{ secrets.BLUESKY_APP_USER }}  
+        password: ${{ secrets.BLUESKY_APP_PASSWORD }}
+        service: bsky.social    
+        retry-count: 1

.github/workflows/python-package.yml

@@ -32,8 +32,7 @@ jobs:
       run: |
         # stop the build if there are Python syntax errors or undefined names
         flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+        flake8 . --count --max-line-length=79 --statistics
     - name: Test with pytest
       run: |
         pytest

.github/workflows/pythonpublish.yml

@@ -26,7 +26,7 @@ jobs:
     - name: Build package
       run: python -m build .
     - name: Publish to Pypi
-      uses: pypa/gh-action-pypi-publish@v1.12.2
+      uses: pypa/gh-action-pypi-publish@v1.13.0
       with:
         password: ${{ secrets.PYPI_APIKEY }}
     - name: Send Bluesky Post

README.md

@@ -5,12 +5,14 @@
 
 # pycmx
 
-The `pycmx` package provides a basic interface for parsing a CMX 3600 EDL and its most most common variations.
+The `pycmx` package provides a basic interface for parsing a CMX 3600 EDL and 
+its most most common variations.
 
 ## Features
 
-* The major variations of the CMX 3600: the standard, "File32" and "File128" 
-  formats are automatically detected and properly read.
+* The major variations of the CMX 3600: the standard, "File32", "File128" and 
+  long Adobe Premiere event numbers are automatically detected and properly
+  read.
 * Preserves relationship between events and individual edits/clips.
 * Remark or comment fields with common recognized forms are read and 
   available to the client, including clip name and source file data.
@@ -83,5 +85,3 @@ Audio channel 7 is present
 >>> events[2].edits[0].channels.video
 False
 ```
-
-

bin/edl2scenelist.py

@@ -12,6 +12,7 @@ logging.basicConfig(format=FORMAT)
 
 log = logging.getLogger(__name__)
 
+
 def all_video_edits(edl):
     for event in edl.events:
         for edit in event.edits:
@@ -30,19 +31,22 @@ def get_scene_name(edit, pattern):
         else:
             return edit.clip_name
 
+
 def output_cmx(outfile, out_list):
     outfile.write("TITLE:  SCENE LIST\r\n")
     outfile.write("FCM: NON-DROP FRAME\r\n")
 
-    for o in out_list:
-        line = "%03i  AX       V     C        00:00:00:00 00:00:00:00 %s %s\r\n" % (0, o['start'],o['end'])
+    for i, o in enumerate(out_list):
+        line = '%03i  AX       V     C        ' % (i)
+        line += '00:00:00:00 00:00:00:00 %s %s\r\n' % (o['start'], o['end'])
         outfile.write(line)
         outfile.write("* FROM CLIP NAME: %s\r\n" % (o['scene']))
 
 
 def output_cols(outfile, out_list):
     for o in out_list:
-        outfile.write("%-12s\t%-12s\t%s\n" % (o['start'], o['end'], o['scene'] ))
+        outfile.write("%-12s\t%-12s\t%s\n" %
+                      (o['start'], o['end'], o['scene']))
 
 
 def scene_list(infile, outfile, out_format, pattern):
@@ -80,22 +84,28 @@ def scene_list(infile, outfile, out_format, pattern):
 
 
 def scene_list_cli():
-    parser = argparse.ArgumentParser(description=
-            'Read video events from an input CMX EDL and output events merged into scenes.')
-    parser.add_argument('-o','--outfile', default=sys.stdout, type=argparse.FileType('w'), 
+    parser = argparse.ArgumentParser(
+        description='Read video events from an input CMX EDL and output '
+        'events merged into scenes.')
+    parser.add_argument('-o', '--outfile', default=sys.stdout,
+                        type=argparse.FileType('w'),
                         help='Output file. Default is stdout.')
     parser.add_argument('-f', '--format', default='cmx', type=str,
-            help='Output format. Options are cols and cmx, cmx is the default.')
+                        help='Output format. Options are cols and cmx, cmx '
+                        'is the default.')
     parser.add_argument('-p', '--pattern', default='V?([A-Z]*[0-9]+)',
-            help='RE pattern for extracting scene name from clip name. The default is "V?([A-Z]*[0-9]+)". ' + \
+                        help='RE pattern for extracting scene name from clip '
+                        'name. The default is "V?([A-Z]*[0-9]+)". ' +
                         'This pattern will be matched case-insensitively.')
-    parser.add_argument('input_edl', default=sys.stdin, type=argparse.FileType('r'), nargs='?', 
+    parser.add_argument('input_edl', default=sys.stdin,
+                        type=argparse.FileType('r'), nargs='?',
                         help='Input file. Default is stdin.')
     args = parser.parse_args()
 
     infile = args.input_edl
 
-    scene_list(infile=infile, outfile=args.outfile , out_format=args.format, pattern=args.pattern)
+    scene_list(infile=infile, outfile=args.outfile,
+               out_format=args.format, pattern=args.pattern)
 
 
 if __name__ == '__main__':

docs/source/conf.py

@@ -12,22 +12,22 @@
 # add these directories to sys.path here. If the directory is relative to the
 # documentation root, use os.path.abspath to make it absolute, like shown here.
 #
+import importlib
 import os
 import sys
 sys.path.insert(0, os.path.abspath('../..'))
 
-import pycmx
 
 # -- Project information -----------------------------------------------------
 
 project = u'pycmx'
-copyright = u'(c) 2023, Jamie Hardt'
+copyright = u'(c) 2025, Jamie Hardt'
 author = u'Jamie Hardt'
 
+release = importlib.metadata.version("pycmx")
+version = release
 # The short X.Y version
-version = pycmx.__version__
 # The full version, including alpha/beta/rc tags
-release = pycmx.__version__
 
 
 # -- General configuration ---------------------------------------------------

pycmx/__init__.py

@@ -2,13 +2,11 @@
 """
 pycmx is a parser for CMX 3600-style EDLs.
 
-This module (c) 2023 Jamie Hardt. For more information on your rights to
+This module (c) 2025 Jamie Hardt. For more information on your rights to
 copy and reuse this software, refer to the LICENSE file included with the
 distribution.
 """
 
-__version__ = '1.2.2'
-
 from .parse_cmx_events import parse_cmx3600
 from .transition import Transition
 from .event import Event

pycmx/channel_map.py

@@ -4,6 +4,7 @@
 from re import (compile, match)
 from typing import Dict, Tuple, Generator
 
+
 class ChannelMap:
     """
     Represents a set of all the channels to which an event applies.
@@ -110,4 +111,3 @@ class ChannelMap:
         out_a = self._audio_channel_set | other._audio_channel_set
 
         return ChannelMap(v=out_v, audio_channels=out_a)
-

pycmx/edit.py

@@ -7,12 +7,16 @@ from .channel_map import ChannelMap
 
 from typing import Optional
 
+
 class Edit:
     """
     An individual source-to-record operation, with a source roll, source and
     recorder timecode in and out, a transition and channels.
     """
-    def __init__(self, edit_statement, audio_ext_statement, clip_name_statement, source_file_statement, trans_name_statement = None):
+
+    def __init__(self, edit_statement, audio_ext_statement,
+                 clip_name_statement, source_file_statement,
+                 trans_name_statement=None):
         self.edit_statement = edit_statement
         self.audio_ext = audio_ext_statement
         self.clip_name_statement = clip_name_statement
@@ -35,7 +39,7 @@ class Edit:
         """
         cm = ChannelMap()
         cm._append_event(self.edit_statement.channels)
-        if self.audio_ext != None:
+        if self.audio_ext is not None:
             cm._append_ext(self.audio_ext)
         return cm
 
@@ -45,9 +49,12 @@ class Edit:
         Get the :obj:`Transition` object associated with this edit.
         """
         if self.trans_name_statement:
-            return Transition(self.edit_statement.trans, self.edit_statement.trans_op, self.trans_name_statement.name)
+            return Transition(self.edit_statement.trans,
+                              self.edit_statement.trans_op,
+                              self.trans_name_statement.name)
         else:
-            return Transition(self.edit_statement.trans, self.edit_statement.trans_op, None)
+            return Transition(self.edit_statement.trans,
+                              self.edit_statement.trans_op, None)
 
     @property
     def source_in(self) -> str:
@@ -124,5 +131,3 @@ class Edit:
             return None
         else:
             return self.clip_name_statement.name
-
-

pycmx/edit_list.py

@@ -1,16 +1,20 @@
 # pycmx
 # (c) 2018 Jamie Hardt
 
-from .parse_cmx_statements import (StmtUnrecognized, StmtFCM, StmtEvent, StmtSourceUMID)
+from .parse_cmx_statements import (
+    StmtUnrecognized, StmtEvent, StmtSourceUMID)
 from .event import Event
 from .channel_map import ChannelMap
 
 from typing import Generator
 
+
 class EditList:
     """
-    Represents an entire edit decision list as returned by :func:`~pycmx.parse_cmx3600()`.
+    Represents an entire edit decision list as returned by
+    :func:`~pycmx.parse_cmx3600()`.
     """
+
     def __init__(self, statements):
         self.title_statement = statements[0]
         self.event_statements = statements[1:]
@@ -20,8 +24,11 @@ class EditList:
         """
         The detected format of the EDL. Possible values are: "3600", "File32",
         "File128", and "unknown".
+
+        Adobe EDLs with more than 999 events will be reported as "3600".
         """
-        first_event = next( (s for s in self.event_statements if type(s) is StmtEvent), None)
+        first_event = next(
+            (s for s in self.event_statements if type(s) is StmtEvent), None)
 
         if first_event:
             if first_event.format == 8:
@@ -35,7 +42,6 @@ class EditList:
         else:
             return 'unknown'
 
-
     @property
     def channels(self) -> ChannelMap:
         """
@@ -49,7 +55,6 @@ class EditList:
 
         return retval
 
-
     @property
     def title(self) -> str:
         """
@@ -57,9 +62,9 @@ class EditList:
         """
         return self.title_statement.title
 
-    
     @property
-    def unrecognized_statements(self) -> Generator[StmtUnrecognized, None, None]:
+    def unrecognized_statements(self) -> Generator[StmtUnrecognized,
+                                                   None, None]:
         """
         A generator for all the unrecognized statements in the list.
         """
@@ -67,17 +72,13 @@ class EditList:
             if type(s) is StmtUnrecognized:
                 yield s
 
-
     @property
     def events(self) -> Generator[Event, None, None]:
         'A generator for all the events in the edit list'
-        is_drop = None
         current_event_num = None
         event_statements = []
         for stmt in self.event_statements:
-            if type(stmt) is StmtFCM:
-                is_drop = stmt.drop
-            elif type(stmt) is StmtEvent:
+            if type(stmt) is StmtEvent:
                 if current_event_num is None:
                     current_event_num = stmt.event
                     event_statements.append(stmt)
@@ -105,5 +106,3 @@ class EditList:
         for stmt in self.event_statements:
             if type(stmt) is StmtSourceUMID:
                 yield stmt
-
-

pycmx/event.py

@@ -1,15 +1,18 @@
 # pycmx
 # (c) 2023 Jamie Hardt
 
-from .parse_cmx_statements import (StmtEvent, StmtClipName, StmtSourceFile, StmtAudioExt, StmtUnrecognized, StmtEffectsName)
+from .parse_cmx_statements import (
+    StmtEvent, StmtClipName, StmtSourceFile, StmtAudioExt, StmtUnrecognized,
+    StmtEffectsName)
 from .edit import Edit
 
 from typing import List, Generator, Optional, Tuple, Any
 
+
 class Event:
     """
-    Represents a collection of :class:`~pycmx.edit.Edit` s, all with the same event number.
-    """
+    Represents a collection of :class:`~pycmx.edit.Edit` s, all with the same
+    event number. """
 
     def __init__(self, statements):
         self.statements = statements
@@ -71,10 +74,12 @@ class Event:
                      audio_ext_statement=e1[1],
                      clip_name_statement=n1,
                      source_file_statement=s1,
-                      trans_name_statement=u1) for (e1,n1,s1,u1) in zip(*the_zip) ]
+                     trans_name_statement=u1)
+                for (e1, n1, s1, u1) in zip(*the_zip)]
 
     @property
-    def unrecognized_statements(self) -> Generator[StmtUnrecognized, None, None]:
+    def unrecognized_statements(self) -> Generator[StmtUnrecognized, None,
+                                                   None]:
         """
         A generator for all the unrecognized statements in the event.
         """
@@ -94,11 +99,10 @@ class Event:
     def _source_file_statements(self) -> List[StmtSourceFile]:
         return [s for s in self.statements if type(s) is StmtSourceFile]
 
-    def _statements_with_audio_ext(self) -> Generator[Tuple[StmtEvent, Optional[StmtAudioExt]], None, None]:
+    def _statements_with_audio_ext(self) -> Generator[
+            Tuple[StmtEvent, Optional[StmtAudioExt]], None, None]:
         for (s1, s2) in zip(self.statements, self.statements[1:]):
             if type(s1) is StmtEvent and type(s2) is StmtAudioExt:
                 yield (s1, s2)
             elif type(s1) is StmtEvent:
                 yield (s1, None)
-
- 

pycmx/parse_cmx_events.py

@@ -8,13 +8,13 @@ from .edit_list import EditList
 
 from typing import TextIO
 
-def parse_cmx3600(f: TextIO):
+
+def parse_cmx3600(f: TextIO) -> EditList:
     """
     Parse a CMX 3600 EDL.
 
-    :param TextIO f: a file-like object, anything that's readlines-able.
+    :param TextIO f: a file-like object, an opened CMX 3600 .EDL file.
     :returns: An :class:`pycmx.edit_list.EditList`.
     """
     statements = parse_cmx3600_statements(f)
     return EditList(statements)
-    

pycmx/parse_cmx_statements.py

@@ -2,9 +2,7 @@
 # (c) 2018 Jamie Hardt
 
 import re
-import sys
 from collections import namedtuple
-from itertools import count
 from typing import TextIO, List
 
 
@@ -12,8 +10,10 @@ from .util import collimate
 
 StmtTitle = namedtuple("Title", ["title", "line_number"])
 StmtFCM = namedtuple("FCM", ["drop", "line_number"])
-StmtEvent =     namedtuple("Event",["event","source","channels","trans",\
-        "trans_op","source_in","source_out","record_in","record_out","format","line_number"])
+StmtEvent = namedtuple("Event", ["event", "source", "channels", "trans",
+                                 "trans_op", "source_in", "source_out",
+                                 "record_in", "record_out", "format",
+                                 "line_number"])
 StmtAudioExt = namedtuple("AudioExt", ["audio3", "audio4", "line_number"])
 StmtClipName = namedtuple("ClipName", ["name", "affect", "line_number"])
 StmtSourceFile = namedtuple("SourceFile", ["filename", "line_number"])
@@ -21,7 +21,8 @@ StmtRemark =    namedtuple("Remark",["text","line_number"])
 StmtEffectsName = namedtuple("EffectsName", ["name", "line_number"])
 StmtSourceUMID = namedtuple("Source", ["name", "umid", "line_number"])
 StmtSplitEdit = namedtuple("SplitEdit", ["video", "magnitude", "line_number"])
-StmtMotionMemory = namedtuple("MotionMemory",["source","fps"]) # FIXME needs more fields
+StmtMotionMemory = namedtuple(
+    "MotionMemory", ["source", "fps"])  # FIXME needs more fields
 StmtUnrecognized = namedtuple("Unrecognized", ["content", "line_number"])
 
 
@@ -30,11 +31,11 @@ def parse_cmx3600_statements(file: TextIO) -> List[object]:
     Return a list of every statement in the file argument.
     """
     lines = file.readlines()
-    line_numbers = count() 
-    return [_parse_cmx3600_line(line.strip(), line_number) \
-            for (line, line_number) in zip(lines,line_numbers)]
+    return [_parse_cmx3600_line(line.strip(), line_number)
+            for (line_number, line) in enumerate(lines)]
 
-def _edl_column_widths(event_field_length, source_field_length):
+
+def _edl_column_widths(event_field_length, source_field_length) -> List[int]:
     return [event_field_length, 2, source_field_length, 1,
             4, 2,  # chans
             4, 1,  # trans
@@ -44,29 +45,31 @@ def _edl_column_widths(event_field_length, source_field_length):
             11, 1,
             11]
 
-def _edl_m2_column_widths():
-    return [2, # "M2"
-            3,3, #
-            8,8,1,4,2,1,4,13,3,1,1]
+# def _edl_m2_column_widths():
+#     return [2, # "M2"
+#             3,3, #
+#             8,8,1,4,2,1,4,13,3,1,1]
 
 
-def _parse_cmx3600_line(line, line_number):
-    long_event_num_p  = re.compile("^[0-9]{6} ")
-    short_event_num_p = re.compile("^[0-9]{3} ")
+def _parse_cmx3600_line(line: str, line_number: int) -> object:
+    """
+    Parses a single CMX EDL line.
+
+    :param line: A single EDL line.
+    :param line_number: The index of this line in the file.
+    """
+    event_num_p = re.compile(r"^(\d+)  ")
+    line_matcher = event_num_p.match(line)
 
-    if isinstance(line,str):
     if line.startswith("TITLE:"):
         return _parse_title(line, line_number)
     elif line.startswith("FCM:"):
         return _parse_fcm(line, line_number)
-        elif long_event_num_p.match(line) != None:
-            length_file_128 = sum(_edl_column_widths(6,128))
-            if len(line) < length_file_128:
-                return _parse_long_standard_form(line, 32, line_number)
-            else:
-                return _parse_long_standard_form(line, 128, line_number)
-        elif short_event_num_p.match(line) != None:
-            return _parse_standard_form(line, line_number)
+    elif line_matcher is not None:
+        event_field_len = len(line_matcher.group(1))
+        source_field_len = len(line) - (event_field_len + 65)
+        return _parse_columns_for_standard_form(line, event_field_len,
+                                                source_field_len, line_number)
     elif line.startswith("AUD"):
         return _parse_extended_audio_channels(line, line_number)
     elif line.startswith("*"):
@@ -83,48 +86,49 @@ def _parse_cmx3600_line(line, line_number):
         return _parse_unrecognized(line, line_number)
 
 
-def _parse_title(line, line_num):
+def _parse_title(line, line_num) -> StmtTitle:
     title = line[6:].strip()
     return StmtTitle(title=title, line_number=line_num)
 
-def _parse_fcm(line, line_num):
+
+def _parse_fcm(line, line_num) -> StmtFCM:
     val = line[4:].strip()
     if val == "DROP FRAME":
         return StmtFCM(drop=True, line_number=line_num)
     else:
         return StmtFCM(drop=False, line_number=line_num)
 
-def _parse_long_standard_form(line,source_field_length, line_number):
-    return _parse_columns_for_standard_form(line, 6, source_field_length, line_number)
-    
-def _parse_standard_form(line, line_number):
-    return _parse_columns_for_standard_form(line, 3, 8, line_number)
 
 def _parse_extended_audio_channels(line, line_number):
     content = line.strip()
-    if content == "AUD   3":
-        return StmtAudioExt(audio3=True, audio4=False, line_number=line_number)
-    elif content == "AUD   4":
-        return StmtAudioExt(audio3=False, audio4=True, line_number=line_number)
-    elif content == "AUD   3     4":
-        return StmtAudioExt(audio3=True, audio4=True, line_number=line_number)
+    audio3 = True if "3" in content else False
+    audio4 = True if "4" in content else False
+
+    if audio3 or audio4:
+        return StmtAudioExt(audio3, audio4, line_number)
     else:
-        return StmtUnrecognized(content=line, line_number=line_number)
+        return StmtUnrecognized(line, line_number)
+
 
 def _parse_remark(line, line_number) -> object:
     if line.startswith("FROM CLIP NAME:"):
-        return StmtClipName(name=line[15:].strip() , affect="from", line_number=line_number)
+        return StmtClipName(name=line[15:].strip(), affect="from",
+                            line_number=line_number)
     elif line.startswith("TO CLIP NAME:"):
-        return StmtClipName(name=line[13:].strip(), affect="to", line_number=line_number)
+        return StmtClipName(name=line[13:].strip(), affect="to",
+                            line_number=line_number)
     elif line.startswith("SOURCE FILE:"):
-        return StmtSourceFile(filename=line[12:].strip() , line_number=line_number)
+        return StmtSourceFile(filename=line[12:].strip(),
+                              line_number=line_number)
     else:
         return StmtRemark(text=line, line_number=line_number)
 
+
 def _parse_effects_name(line, line_number) -> StmtEffectsName:
     name = line[16:].strip()
     return StmtEffectsName(name=name, line_number=line_number)
 
+
 def _parse_split(line, line_number):
     split_type = line[10:21]
     is_video = False
@@ -132,7 +136,8 @@ def _parse_split(line, line_number):
         is_video = True
 
     split_mag = line[24:35]
-    return StmtSplitEdit(video=is_video, magnitude=split_mag, line_number=line_number)
+    return StmtSplitEdit(video=is_video, magnitude=split_mag,
+                         line_number=line_number)
 
 
 def _parse_motion_memory(line, line_number):
@@ -142,7 +147,9 @@ def _parse_motion_memory(line, line_number):
 def _parse_unrecognized(line, line_number):
     return StmtUnrecognized(content=line, line_number=line_number)
 
-def _parse_columns_for_standard_form(line, event_field_length, source_field_length, line_number):
+
+def _parse_columns_for_standard_form(line, event_field_length,
+                                     source_field_length, line_number):
     col_widths = _edl_column_widths(event_field_length, source_field_length)
 
     if sum(col_widths) > len(line):
@@ -159,11 +166,9 @@ def _parse_columns_for_standard_form(line, event_field_length, source_field_leng
                      source_out=column_strings[12].strip(),
                      record_in=column_strings[14].strip(),
                      record_out=column_strings[16].strip(),
-		    line_number=line_number,
-                    format=source_field_length)
+                     line_number=line_number, format=source_field_length)
 
 
 def _parse_source_umid_statement(line, line_number):
-    trimmed = line[3:].strip()
+    # trimmed = line[3:].strip()
     return StmtSourceUMID(name=None, umid=None, line_number=line_number)
-

pycmx/transition.py

@@ -3,6 +3,7 @@
 
 from typing import Optional
 
+
 class Transition:
     """
     A CMX transition: a wipe, dissolve or cut.
@@ -86,5 +87,3 @@ class Transition:
         the key foreground and replaced with the key background.
         """
         return self.transition == Transition.KeyOut
-
-

pycmx/util.py

@@ -28,4 +28,3 @@ def collimate(a_string, column_widths):
     element = a_string[:width]
     rest = a_string[width:]
     return [element] + collimate(rest, column_widths[1:])
-

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pycmx"
-version = "1.2.3"
+version = "1.4.0"
 description = "Python CMX 3600 Edit Decision List Parser"
 authors = ["Jamie Hardt <jamiehardt@me.com>"]
 license = "MIT"

tests/edls/ISSUE_14_conform_edl_issue_01.edl

@@ -0,0 +1,13 @@
+TITLE: conform_edl_issue_01
+FCM: NON-DROP FRAME
+
+001  C_0022C003_241016_092821_h1F4X V     C        09:33:31:12 09:33:33:14 01:00:06:20 01:00:08:22  
+* FROM CLIP NAME: 13A-1-C
+
+002  B_0020C009_241003_214837_h1C2T V     C        21:48:54:22 21:48:55:15 01:01:34:06 01:01:34:23  
+* FROM CLIP NAME: 111B-1-B
+
+003  B_0088C002_241125_144410_h1C2T V     C        13:48:57:10 13:48:58:11 01:01:41:13 01:01:42:14  
+M2   B_0088C002_241125_144410_h1C2T       031.7                13:48:57:10
+* FROM CLIP NAME: 102C-2-B
+

tests/edls/ISSUE_14_conform_edl_issue_02.edl

@@ -0,0 +1,33 @@
+TITLE: conform_edl_issue_02
+FCM: NON-DROP FRAME
+
+001  C_0019C005_241014_204338_h1F4X V     C        11:19:41:07 11:19:49:03 02:04:48:10 02:04:56:06  
+* FROM CLIP NAME: 40E-4 MOS*
+
+002  B_0075C004_241114_164247_h1C2T V     C        15:43:03:12 15:43:05:17 02:06:03:03 02:06:05:08  
+* FROM CLIP NAME: 39B-4-B
+
+003  A_0079C015_241112_160227_h1EHP V     C        16:06:21:10 16:06:23:03 02:08:14:07 02:08:16:00  
+* FROM CLIP NAME: 46L-2
+
+004  C_0047C005_241121_123629_h1F4X V     C        11:41:15:17 11:41:21:01 02:09:08:01 02:09:13:09  
+* FROM CLIP NAME: 49-5-C
+
+005  A_0003C002_240923_130856_h1EHP V     C        14:11:06:01 14:11:12:03 02:11:26:11 02:11:32:13  
+* FROM CLIP NAME: 54-2-A
+
+006  A_0090C003_241119_095341_h1EHP V     C        09:57:25:14 09:57:32:20 02:13:40:13 02:13:47:19  
+* FROM CLIP NAME: 57B-3-A*
+
+007  A_0090C008_241119_102624_h1EHP V     C        10:30:43:07 10:30:46:23 02:14:16:01 02:14:19:17  
+* FROM CLIP NAME: 57D-2*
+
+008  B_0079C003_241119_105658_h1C2T V     C        09:58:57:15 09:59:00:14 02:15:01:08 02:15:04:07  
+* FROM CLIP NAME: 57B-3-B*
+
+009  A_0005C011_240924_113730_h1EHP V     C        12:40:01:07 12:40:04:06 02:16:13:11 02:16:16:10  
+* FROM CLIP NAME: 58C-3-A
+
+010  A_0060C001_241030_133415_h1EHP V     C        13:35:38:11 13:35:41:17 02:18:29:17 02:18:32:23  
+* FROM CLIP NAME: 61A-1
+

tests/edls/ISSUE_14_conform_edl_issue_03.edl

@@ -0,0 +1,129 @@
+TITLE: conform_edl_issue_03
+FCM: NON-DROP FRAME
+
+002  A_0113C007_250602_103141_h1D4P V     C        10:32:32:22 10:32:48:16 01:00:20:23 01:00:36:17  
+* FROM CLIP NAME: AP002A-3*
+
+003  A_0113C004_250602_101043_h1D4P V     C        10:11:34:01 10:11:48:01 01:00:36:17 01:00:50:17  
+* FROM CLIP NAME: AP002-4*
+
+004  A_0113C018_250602_122238_h1D4P V     C        12:24:03:12 12:24:04:17 01:01:15:17 01:01:16:22  
+* FROM CLIP NAME: AP002E-3-A*
+
+005  A_0113C018_250602_122238_h1D4P V     C        12:24:05:22 12:24:18:02 01:01:20:21 01:01:33:01  
+* FROM CLIP NAME: AP002E-3-A*
+
+006  A_0113C022_250602_125451_h1D4P V     C        12:55:43:09 12:55:55:03 01:01:33:01 01:01:44:19  
+* FROM CLIP NAME: AP002F-4-A*
+
+007  A_0113C024_250602_132048_h1D4P V     C        13:21:54:17 13:22:00:07 01:01:44:19 01:01:50:09  
+* FROM CLIP NAME: AP002G-2-A
+
+008  B_0098C009_250602_115121_h1EZ3 V     C        12:56:02:03 12:56:05:11 01:01:50:09 01:01:53:17  
+* FROM CLIP NAME: AP002F-4-B*
+
+009  A_0113C024_250602_132048_h1D4P V     C        13:21:41:20 13:21:46:01 01:01:53:17 01:01:57:22  
+* FROM CLIP NAME: AP002G-2-A
+
+010  A_0113C022_250602_125451_h1D4P V     C        12:56:08:09 12:56:12:23 01:01:57:22 01:02:02:12  
+* FROM CLIP NAME: AP002F-4-A*
+
+011  B_0099C004_250602_133201_h1EZ3 V     C        14:36:34:09 14:36:38:03 01:02:06:07 01:02:10:01  
+* FROM CLIP NAME: AP002H-4-B*
+
+012  A_0114C009_250602_161406_h1D4P V     C        16:15:15:09 16:15:17:16 01:02:10:01 01:02:12:08  
+* FROM CLIP NAME: AP002M-1-A
+
+013  A_0115C001_250602_172408_h1D4P V     C        17:25:55:01 17:25:59:17 01:02:20:15 01:02:25:07  
+* FROM CLIP NAME: AP002Q-1-A
+
+014  B_0099C019_250602_153856_h1EZ3 V     C        16:43:35:06 16:43:38:13 01:02:25:07 01:02:28:14  
+* FROM CLIP NAME: AP002N-2
+
+015  A_0115C002_250602_172803_h1D4P V     C        17:29:28:01 17:29:32:02 01:02:28:14 01:02:32:15  
+* FROM CLIP NAME: AP002Q-2-A*
+
+016  B_0099C019_250602_153856_h1EZ3 V     C        16:43:41:10 16:43:45:12 01:02:32:15 01:02:36:17  
+* FROM CLIP NAME: AP002N-2
+
+017  A_0115C001_250602_172408_h1D4P V     C        17:26:08:05 17:26:10:20 01:02:36:17 01:02:39:08  
+* FROM CLIP NAME: AP002Q-1-A
+
+018  B_0099C018_250602_153023_h1EZ3 V     C        16:35:28:00 16:35:30:10 01:02:39:08 01:02:41:18  
+* FROM CLIP NAME: AP002N-1*
+
+019  B_0100C001_250602_162041_h1EZ3 V     C        17:26:17:05 17:26:24:23 01:02:41:18 01:02:49:12  
+* FROM CLIP NAME: AP002Q-1-B
+
+020  A_0114C010_250602_162156_h1D4P V     C        16:24:28:03 16:24:33:20 01:02:49:12 01:02:55:05  
+* FROM CLIP NAME: AP002M-2-A*
+
+021  B_0099C007_250602_140229_h1EZ3 V     C        15:07:55:18 15:08:00:01 01:02:55:05 01:02:59:12  
+* FROM CLIP NAME: AP002K-2-B
+
+022  B_0099C018_250602_153023_h1EZ3 V     C        16:35:47:23 16:35:49:05 01:02:59:12 01:03:00:18  
+* FROM CLIP NAME: AP002N-1*
+
+023  B_0099C007_250602_140229_h1EZ3 V     C        15:08:01:17 15:08:12:19 01:03:00:18 01:03:11:20  
+* FROM CLIP NAME: AP002K-2-B
+
+024  B_0099C018_250602_153023_h1EZ3 V     C        16:36:10:00 16:36:11:10 01:03:11:20 01:03:13:06  
+* FROM CLIP NAME: AP002N-1*
+
+025  B_0099C007_250602_140229_h1EZ3 V     C        15:08:15:08 15:08:22:13 01:03:13:06 01:03:20:11  
+* FROM CLIP NAME: AP002K-2-B
+
+026  B_0099C018_250602_153023_h1EZ3 V     C        16:36:18:21 16:36:23:01 01:03:20:11 01:03:24:15  
+* FROM CLIP NAME: AP002N-1*
+
+027  B_0099C007_250602_140229_h1EZ3 V     C        15:08:33:21 15:08:37:23 01:03:30:14 01:03:34:16  
+* FROM CLIP NAME: AP002K-2-B
+
+028  B_0099C017_250602_151824_h1EZ3 V     C        16:25:21:01 16:25:29:04 01:03:34:16 01:03:42:19  
+* FROM CLIP NAME: AP002M-2-B*
+
+029  B_0099C007_250602_140229_h1EZ3 V     C        15:08:47:10 15:08:55:02 01:03:42:19 01:03:50:11  
+* FROM CLIP NAME: AP002K-2-B
+
+030  B_0099C017_250602_151824_h1EZ3 V     C        16:25:39:00 16:25:42:03 01:03:50:11 01:03:53:14  
+* FROM CLIP NAME: AP002M-2-B*
+
+031  B_0099C007_250602_140229_h1EZ3 V     C        15:08:58:10 15:09:03:04 01:03:53:14 01:03:58:08  
+* FROM CLIP NAME: AP002K-2-B
+
+032  B_0099C016_250602_151038_h1EZ3 V     C        16:17:20:21 16:17:23:10 01:03:58:08 01:04:00:21  
+* FROM CLIP NAME: AP002M-1-B
+
+033  B_0099C007_250602_140229_h1EZ3 V     C        15:09:05:09 15:09:16:04 01:04:00:21 01:04:11:16  
+* FROM CLIP NAME: AP002K-2-B
+
+034  B_0099C019_250602_153856_h1EZ3 V     C        16:45:46:08 16:45:52:18 01:04:11:16 01:04:18:02  
+* FROM CLIP NAME: AP002N-2
+
+035  B_0099C007_250602_140229_h1EZ3 V     C        15:09:24:21 15:09:27:22 01:04:18:02 01:04:21:03  
+* FROM CLIP NAME: AP002K-2-B
+
+036  B_0099C019_250602_153856_h1EZ3 V     C        16:45:55:19 16:45:58:02 01:04:21:03 01:04:23:10  
+* FROM CLIP NAME: AP002N-2
+
+037  B_0099C004_250602_133201_h1EZ3 V     C        14:38:53:22 14:38:58:19 01:04:23:10 01:04:28:07  
+* FROM CLIP NAME: AP002H-4-B*
+
+038  B_0099C015_250602_150046_h1EZ3 V     C        16:07:52:23 16:07:55:04 01:04:28:07 01:04:30:12  
+* FROM CLIP NAME: AP002L-1-B
+
+039  B_0099C007_250602_140229_h1EZ3 V     C        15:09:35:22 15:09:40:19 01:04:30:12 01:04:35:09  
+* FROM CLIP NAME: AP002K-2-B
+
+040  B_0099C015_250602_150046_h1EZ3 V     C        16:08:01:21 16:08:05:01 01:04:35:09 01:04:38:13  
+* FROM CLIP NAME: AP002L-1-B
+
+041  B_0099C007_250602_140229_h1EZ3 V     C        15:09:45:16 15:09:55:00 01:04:38:13 01:04:47:21  
+* FROM CLIP NAME: AP002K-2-B
+
+042  M018C0005_240925_1F4L13 V     C        18:44:20:12 18:44:23:18 01:04:50:16 01:04:53:22  
+* FROM CLIP NAME: 13-1-SER-1-M MOS
+
+043  A_0022C009_241003_141208_h1EHP V     C        15:13:54:11 15:13:56:04 01:12:22:02 01:12:23:19  
+* FROM CLIP NAME: 26H-3-A*
+

tests/test_parse.py

@@ -2,6 +2,7 @@ from unittest import TestCase
 
 import pycmx
 
+
 class TestParse(TestCase):
 
     files = ["INS4_R1_010417.edl",
@@ -22,7 +23,8 @@ class TestParse(TestCase):
                 edl = pycmx.parse_cmx3600(f)
                 actual = len(list(edl.events))
                 self.assertTrue(actual == count,
-                        "expected %i in file %s but found %i" % (count, fn, actual))
+                                "expected %i in file %s but found %i"
+                                % (count, fn, actual))
 
     def test_list_sanity(self):
         for fn in type(self).files:
@@ -31,17 +33,16 @@ class TestParse(TestCase):
                 self.assertTrue(type(edl.title) is str)
                 self.assertTrue(len(edl.title) > 0)
 
-
     def test_event_sanity(self):
         for fn in type(self).files:
             path = "tests/edls/" + fn
             with open(path, 'r') as f:
                 edl = pycmx.parse_cmx3600(f)
                 for index, event in enumerate(edl.events):
-                    self.assertTrue( len(event.edits) > 0 )
-                    self.assertTrue( event.number == index + 1 )
-
-
+                    self.assertTrue(len(event.edits) > 0,
+                                    f"Failed for {path}")
+                    self.assertEqual(event.number, index + 1,
+                                     f"Failed for {path}")
 
     def test_events(self):
         with open("tests/edls/TEST.edl", 'r') as f:
@@ -51,12 +52,14 @@ class TestParse(TestCase):
             self.assertEqual(events[0].number, 1)
             self.assertEqual(events[0].edits[0].source, "OY_HEAD_")
             self.assertEqual(events[0].edits[0].clip_name, "HEAD LEADER MONO")
-            self.assertEqual( events[0].edits[0].source_file , "OY_HEAD_LEADER.MOV")
+            self.assertEqual(
+                events[0].edits[0].source_file, "OY_HEAD_LEADER.MOV")
             self.assertEqual(events[0].edits[0].source_in, "00:00:00:00")
             self.assertEqual(events[0].edits[0].source_out, "00:00:00:00")
             self.assertEqual(events[0].edits[0].record_in, "01:00:00:00")
             self.assertEqual(events[0].edits[0].record_out, "01:00:08:00")
-            self.assertTrue( events[0].edits[0].transition.kind == pycmx.Transition.Cut)
+            self.assertTrue(
+                events[0].edits[0].transition.kind == pycmx.Transition.Cut)
 
     def test_channel_map(self):
         with open("tests/edls/TEST.edl", 'r') as f:
@@ -68,7 +71,6 @@ class TestParse(TestCase):
             self.assertTrue(events[2].edits[0].channels.get_audio_channel(7))
             self.assertTrue(events[2].edits[0].channels.audio)
 
-
     def test_multi_edit_events(self):
         with open("tests/edls/TEST.edl", 'r') as f:
             edl = pycmx.parse_cmx3600(f)
@@ -78,20 +80,25 @@ class TestParse(TestCase):
             self.assertEqual(len(events[42].edits), 2)
 
             self.assertEqual(events[42].edits[0].source, "TC_R1_V1")
-            self.assertEqual( events[42].edits[0].clip_name , "TC R1 V1.2 TEMP1 FX ST.WAV")
+            self.assertEqual(events[42].edits[0].clip_name,
+                             "TC R1 V1.2 TEMP1 FX ST.WAV")
             self.assertEqual(events[42].edits[0].source_in, "00:00:00:00")
             self.assertEqual(events[42].edits[0].source_out, "00:00:00:00")
             self.assertEqual(events[42].edits[0].record_in, "01:08:56:09")
             self.assertEqual(events[42].edits[0].record_out, "01:08:56:09")
-            self.assertTrue( events[42].edits[0].transition.kind == pycmx.Transition.Cut)
+            self.assertTrue(
+                events[42].edits[0].transition.kind == pycmx.Transition.Cut)
 
             self.assertEqual(events[42].edits[1].source, "TC_R1_V6")
-            self.assertEqual( events[42].edits[1].clip_name , "TC R1 V6 TEMP2 ST FX.WAV")
+            self.assertEqual(events[42].edits[1].clip_name,
+                             "TC R1 V6 TEMP2 ST FX.WAV")
             self.assertEqual(events[42].edits[1].source_in, "00:00:00:00")
             self.assertEqual(events[42].edits[1].source_out, "00:00:00:00")
             self.assertEqual(events[42].edits[1].record_in, "01:08:56:09")
             self.assertEqual(events[42].edits[1].record_out, "01:08:56:11")
-            self.assertTrue( events[42].edits[1].transition.kind == pycmx.Transition.Dissolve)
+            self.assertTrue(
+                events[42].edits[1].transition.kind ==
+                pycmx.Transition.Dissolve)
 
     def test_line_numbers(self):
         with open("tests/edls/ToD_R4_LOCK3.1_030618_Video.edl") as f:
@@ -106,8 +113,27 @@ class TestParse(TestCase):
         with open("tests/edls/test_25.edl", "r") as f:
             edl = pycmx.parse_cmx3600(f)
             events = list(edl.events)
-            self.assertEqual( events[4].edits[1].transition.name ,  "CROSS DISSOLVE" )
+            self.assertEqual(
+                events[4].edits[1].transition.name,  "CROSS DISSOLVE")
 
+    def test_adobe_wide(self):
+        with open("tests/edls/adobe_dai109_test.txt", 'r',
+                  encoding='ISO-8859-1') as f:
+            edl = pycmx.parse_cmx3600(f)
+            events = list(edl.events)
 
-    # add test for edit_list.channels
+            self.assertEqual(len(events), 2839)
 
+    def test_issue14(self):
+        with open("tests/edls/ISSUE_14_conform_edl_issue_03.edl", "r") as f:
+            edl = pycmx.parse_cmx3600(f)
+
+            for event in edl.events:
+                if event.number == 42:
+                    self.assertEqual(len(event.edits), 1)
+                    self.assertEqual(event.edits[0].source,
+                                     "M018C0005_240925_1F4L13")
+                    self.assertEqual(event.edits[0].transition.kind,
+                                     pycmx.Transition.Cut)
+                    self.assertEqual(event.edits[0].source_in,
+                                     "18:44:20:12")