Bumped version and tweaked Bluesky message

Sprucing-up Build System and Workflows

.github/workflows/python-package.yml

@@ -0,0 +1,39 @@
+# This workflow will install Python dependencies, run tests and lint with a variety of Python versions
+# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python
+
+name: Lint and Test
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v4.3.0
+      with:
+        python-version: ${{ matrix.python-version }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        python -m pip install flake8 pytest
+        python -m pip install -e .
+    - name: Lint with flake8
+      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
+    - name: Test with pytest
+      run: |
+        pytest

.github/workflows/pythonpublish.yml

@@ -0,0 +1,42 @@
+name: Upload Python Package
+
+on:
+  release:
+    types: [published]
+    
+permissions:
+  contents: read
+  id-token: write
+  
+jobs:
+  deploy:
+    runs-on: ubuntu-latest
+    environment:
+      name: release
+    steps:
+    - uses: actions/checkout@v4.2.2
+    - name: Set up Python
+      uses: actions/setup-python@v5.3.0
+      with:
+        python-version: '3.x'
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install build
+    - name: Build package
+      run: python -m build .
+    - name: Publish to Pypi
+      uses: pypa/gh-action-pypi-publish@v1.12.2
+      with:
+        password: ${{ secrets.PYPI_APIKEY }}
+    - name: Send Bluesky Post
+      uses: myConsciousness/bluesky-post@v5
+      with:
+        text: |
+          I've released a new version of pycmx, my python module for 
+          reading CMX EDLs.
+        link-preview-url: ${{ github.server_url }}/${{ github.repository }}
+        identifier: ${{ secrets.BLUESKY_APP_USER }}  
+        password: ${{ secrets.BLUESKY_APP_PASSWORD }}
+        service: bsky.social    
+        retry-count: 1

.gitignore

@@ -10,3 +10,13 @@
 
 # Vim Swapfiles
 *.swp
+.DS_Store
+
+venv/
+
+.coverage
+lcov.info
+
+
+# venv
+venv/

.readthedocs.yaml

@@ -0,0 +1,32 @@
+# .readthedocs.yaml
+# Read the Docs configuration file
+# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
+
+# Required
+version: 2
+
+# Set the version of Python and other tools you might need
+build:
+  os: ubuntu-20.04
+  tools:
+    python: "3.10"
+    # You can also specify other tool versions:
+    # nodejs: "16"
+    # rust: "1.55"
+    # golang: "1.17"
+
+# Build documentation in the docs/ directory with Sphinx
+sphinx:
+   configuration: docs/source/conf.py
+
+#If using Sphinx, optionally build your docs in additional formats such as PDF
+formats:
+   - pdf
+
+#Optionally declare the Python requirements required to build your docs
+python:
+  install:
+    - method: pip
+      path: .
+      extra_requirements:
+        - doc

.travis.yml

@@ -1,7 +0,0 @@
-language: python
-python:
-  - "3.6"
-script:
-  - "python3 setup.py test"
-install:
-  - "pip3 install setuptools"

CONTRIBUTING.md

@@ -0,0 +1,13 @@
+# Contributing
+
+Contributions to this project are welcome!
+
+The best way to contribute code to this project is to find this project on [Github][github] and submit a pull request.
+
+## Call for EDLs
+
+If you have EDLs you are having trouble working with becuase of unusual formatting, please send me a copy! Contact us
+through [Github].
+
+
+[github]: https://github.com/iluvcapra/pycmx

LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2018 Jamie Hardt.
+Copyright (c) 2022 Jamie Hardt.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -1,4 +1,6 @@
-[![Build Status](https://travis-ci.com/iluvcapra/pycmx.svg?branch=master)](https://travis-ci.com/iluvcapra/pycmx) [![Documentation Status](https://readthedocs.org/projects/pycmx/badge/?version=latest)](https://pycmx.readthedocs.io/en/latest/?badge=latest)
+[![Documentation Status](https://readthedocs.org/projects/pycmx/badge/?version=latest)](https://pycmx.readthedocs.io/en/latest/?badge=latest) ![](https://img.shields.io/github/license/iluvcapra/pycmx.svg) ![](https://img.shields.io/pypi/pyversions/pycmx.svg) [![](https://img.shields.io/pypi/v/pycmx.svg)](https://pypi.org/project/pycmx/) ![](https://img.shields.io/pypi/wheel/pycmx.svg)
+![GitHub last commit](https://img.shields.io/github/last-commit/iluvcapra/pycmx)
+[![Lint and Test](https://github.com/iluvcapra/pycmx/actions/workflows/python-package.yml/badge.svg)](https://github.com/iluvcapra/pycmx/actions/workflows/python-package.yml)
 
 
 # pycmx
@@ -82,32 +84,4 @@ Audio channel 7 is present
 False
 ```
 
-## How is this different from `python-edl`?
 
-There are two important differences between `import edl` and `import pycmx` 
-and motivated my development of this module.
-
-1. The `pycmx` parser doesn't take timecode or framerates into account, 
-   and strictly treats timecodes like opaque values. As far as `pycmx` is 
-   concerend, they're just strings. This was done because in my experience, 
-   the frame rate of an EDL is often difficult to precisely determine and 
-   often the frame rate of various sources is different from the frame rate 
-   of the target track.
-   
-   In any event, timecodes in an EDL are a kind of *address* and are not
-   exactly scalar, they're meant to point to a particular block of video or 
-   audio data on a medium and presuming that they refer to a real time, or 
-   duration, or are convertible, etc. isn't always safe.
-
-2. The `pycmx` parser reads event numbers and keeps track of which EDL rows
-   are meant to happen "at the same time," with two decks. This makes it 
-   easier to reconstruct transition A/B clips, and read clip names from
-   such events appropriately.
- 
-## Should I Use This?
-
-At this time, this is (at best) beta software. I feel like the interface is 
-about where where I'd like it to be but more testing is required.
-
-Contributions are welcome and will make this module production-ready all the
-faster! Please reach out or file a ticket! 

docs/make.bat

@@ -1,35 +0,0 @@
-@ECHO OFF
-
-pushd %~dp0
-
-REM Command file for Sphinx documentation
-
-if "%SPHINXBUILD%" == "" (
-	set SPHINXBUILD=sphinx-build
-)
-set SOURCEDIR=source
-set BUILDDIR=build
-
-if "%1" == "" goto help
-
-%SPHINXBUILD% >NUL 2>NUL
-if errorlevel 9009 (
-	echo.
-	echo.The 'sphinx-build' command was not found. Make sure you have Sphinx
-	echo.installed, then set the SPHINXBUILD environment variable to point
-	echo.to the full path of the 'sphinx-build' executable. Alternatively you
-	echo.may add the Sphinx directory to PATH.
-	echo.
-	echo.If you don't have Sphinx installed, grab it from
-	echo.http://sphinx-doc.org/
-	exit /b 1
-)
-
-%SPHINXBUILD% -M %1 %SOURCEDIR% %BUILDDIR% %SPHINXOPTS%
-goto end
-
-:help
-%SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR% %SPHINXOPTS%
-
-:end
-popd

docs/source/classes.rst

@@ -0,0 +1,24 @@
+pycmx Classes
+=============
+
+.. autoclass:: pycmx.edit_list.EditList 
+   :members:
+
+
+.. autoclass:: pycmx.event.Event 
+   :members:
+
+
+.. autoclass:: pycmx.edit.Edit  
+   :members:
+
+
+.. autoclass:: pycmx.transition.Transition
+   :members:
+
+
+.. autoclass:: pycmx.channel_map.ChannelMap
+   :members:
+
+
+

docs/source/conf.py

@@ -14,19 +14,20 @@
 #
 import os
 import sys
-sys.path.insert(0, os.path.abspath('../../pycmx'))
+sys.path.insert(0, os.path.abspath('../..'))
 
+import pycmx
 
 # -- Project information -----------------------------------------------------
 
 project = u'pycmx'
-copyright = u'2018, Jamie Hardt'
+copyright = u'(c) 2023, Jamie Hardt'
 author = u'Jamie Hardt'
 
 # The short X.Y version
-version = u''
+version = pycmx.__version__
 # The full version, including alpha/beta/rc tags
-release = u''
+release = pycmx.__version__
 
 
 # -- General configuration ---------------------------------------------------
@@ -63,7 +64,7 @@ master_doc = 'index'
 #
 # This is also used if you do content translation via gettext catalogs.
 # Usually you set "language" from the command line for these cases.
-language = None
+language = 'em'
 
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.
@@ -144,7 +145,7 @@ latex_documents = [
 # (source start file, name, description, authors, manual section).
 man_pages = [
     (master_doc, 'pycmx', u'pycmx Documentation',
-     [author], 1)
+     [author], "3p")
 ]
 
 

docs/source/function.rst

@@ -0,0 +1,7 @@
+Parse Function
+==============
+
+
+.. autofunction:: pycmx.parse_cmx_events.parse_cmx3600
+
+

docs/source/index.rst

@@ -8,13 +8,13 @@ Welcome to pycmx's documentation!
 
 .. toctree::
    :maxdepth: 5
-   :caption: API Reference:
-   
-   pycmx
+   :caption: API Reference
+
+   function
+   classes
 
 Indices and tables
 ==================
 
 * :ref:`genindex`
-* :ref:`modindex`
 * :ref:`search`

docs/source/modules.rst

@@ -1,7 +0,0 @@
-pycmx
-=====
-
-.. toctree::
-   :maxdepth: 4
-
-   pycmx

docs/source/pycmx.rst

@@ -1,46 +0,0 @@
-pycmx package
-=============
-
-Submodules
-----------
-
-pycmx.channel\_map module
--------------------------
-
-.. automodule:: pycmx.channel_map
-    :members:
-    :undoc-members:
-    :show-inheritance:
-
-pycmx.parse\_cmx\_events module
--------------------------------
-
-.. automodule:: pycmx.parse_cmx_events
-    :members:
-    :undoc-members:
-    :show-inheritance:
-
-pycmx.parse\_cmx\_statements module
------------------------------------
-
-.. automodule:: pycmx.parse_cmx_statements
-    :members:
-    :undoc-members:
-    :show-inheritance:
-
-pycmx.util module
------------------
-
-.. automodule:: pycmx.util
-    :members:
-    :undoc-members:
-    :show-inheritance:
-
-
-Module contents
----------------
-
-.. automodule:: pycmx
-    :members:
-    :undoc-members:
-    :show-inheritance:

pycmx/__init__.py

@@ -1,16 +1,15 @@
 # -*- coding: utf-8 -*-
 """
-pycmx is a module for parsing CMX 3600-style EDLs. For more information and 
-examples see README.md
+pycmx is a parser for CMX 3600-style EDLs.
 
-This module (c) 2018 Jamie Hardt. For more information on your rights to 
-copy and reuse this software, refer to the LICENSE file included with the 
+This module (c) 2023 Jamie Hardt. For more information on your rights to
+copy and reuse this software, refer to the LICENSE file included with the
 distribution.
 """
 
-__version__ = '0.7'
-__author__  = 'Jamie Hardt'
-
-from .parse_cmx_events import parse_cmx3600, Transition, Event, Edit
-from . import parse_cmx_events
+__version__ = '1.2.2'
 
+from .parse_cmx_events import parse_cmx3600
+from .transition import Transition
+from .event import Event
+from .edit import Edit

pycmx/channel_map.py

@@ -2,14 +2,15 @@
 # (c) 2018 Jamie Hardt
 
 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.
     """
 
-    _chan_map = {     "V" :    (True,   False,   False),
+    _chan_map : Dict[str, Tuple] = {     
+                "V" :    (True,   False,   False),
                 "A" :    (False,  True,    False),
                 "A2" :   (False,  False,   True),
                 "AA" :   (False,  True,    True),
@@ -23,65 +24,70 @@ class ChannelMap:
         self.v = v
 
     @property
-    def video(self):
+    def video(self) -> bool:
         'True if video is included'
         return self.v
 
     @property
-    def channels(self):
+    def audio(self) -> bool:
+        'True if an audio channel is included'
+        return len(self._audio_channel_set) > 0
+
+    @property
+    def channels(self) -> Generator[int, None, None]:
         'A generator for each audio channel'
         for c in self._audio_channel_set:
             yield c
 
     @property
-    def a1(self):
-        """True if A1 is included."""
+    def a1(self) -> bool:
+        """True if A1 is included"""
         return self.get_audio_channel(1)
 
     @a1.setter
-    def a1(self,val):
+    def a1(self, val: bool):
         self.set_audio_channel(1,val)
 
     @property
-    def a2(self):
-        """True if A2 is included."""
+    def a2(self) -> bool:
+        """True if A2 is included"""
         return self.get_audio_channel(2)
 
     @a2.setter
-    def a2(self,val):
+    def a2(self, val: bool):
         self.set_audio_channel(2,val)
 
     @property
-    def a3(self):
-        """True if A3 is included."""
+    def a3(self) -> bool:
+        """True if A3 is included"""
         return self.get_audio_channel(3)
 
     @a3.setter
-    def a3(self,val):
+    def a3(self, val: bool):
         self.set_audio_channel(3,val)
     
     @property
-    def a4(self):
-        """True if A4 is included."""
+    def a4(self) -> bool:
+        """True if A4 is included"""
         return self.get_audio_channel(4)
 
     @a4.setter
-    def a4(self,val):
+    def a4(self,val: bool):
         self.set_audio_channel(4,val)
 
-    def get_audio_channel(self,chan_num):
-        """True if chan_num is included."""
+    def get_audio_channel(self, chan_num) -> bool:
+        """True if chan_num is included"""
         return (chan_num in self._audio_channel_set)
 
-    def set_audio_channel(self,chan_num,enabled):
-        """If enabled is true, chan_num will be included."""
+    def set_audio_channel(self,chan_num, enabled: bool):
+        """If enabled is true, chan_num will be included"""
         if enabled:
             self._audio_channel_set.add(chan_num)
         elif self.get_audio_channel(chan_num):
             self._audio_channel_set.remove(chan_num)
     
     def _append_event(self, event_str):
-        alt_channel_re = compile('^A(\d+)')
+        alt_channel_re = compile(r'^A(\d+)')
         if event_str in self._chan_map:
             channels = self._chan_map[event_str]
             self.v = channels[0]
@@ -92,7 +98,16 @@ class ChannelMap:
             if matchresult:
                 self.set_audio_channel(int( matchresult.group(1)), True )
 
-    def _append_sxt(self, audio_ext):
-        self.a3 = ext.audio3
-        self.a4 = ext.audio4
+    def _append_ext(self, audio_ext):
+        self.a3 = audio_ext.audio3
+        self.a4 = audio_ext.audio4
+
+    def __or__(self, other):
+        """
+        the logical union of this channel map with another
+        """
+        out_v = self.video | other.video
+        out_a = self._audio_channel_set | other._audio_channel_set
+
+        return ChannelMap(v=out_v,audio_channels = out_a)
 

pycmx/edit.py

@@ -0,0 +1,128 @@
+# pycmx
+# (c) 2018 Jamie Hardt
+
+from .transition import Transition
+from .channel_map import ChannelMap
+# from .parse_cmx_statements import StmtEffectsName
+
+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):
+        self.edit_statement = edit_statement
+        self.audio_ext = audio_ext_statement
+        self.clip_name_statement = clip_name_statement
+        self.source_file_statement = source_file_statement
+        self.trans_name_statement = trans_name_statement 
+
+    @property
+    def line_number(self) -> int:
+        """
+        Get the line number for the "standard form" statement associated with
+        this edit. Line numbers a zero-indexed, such that the 
+        "TITLE:" record is line zero.
+        """
+        return self.edit_statement.line_number
+
+    @property
+    def channels(self) -> ChannelMap:
+        """
+        Get the :obj:`ChannelMap` object associated with this Edit.
+        """
+        cm = ChannelMap()
+        cm._append_event(self.edit_statement.channels)
+        if self.audio_ext != None:
+            cm._append_ext(self.audio_ext)
+        return cm
+
+    @property
+    def transition(self) -> Transition:
+        """
+        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)
+        else:
+            return Transition(self.edit_statement.trans, self.edit_statement.trans_op, None)
+    
+    @property
+    def source_in(self) -> str:
+        """
+        Get the source in timecode.
+        """
+        return self.edit_statement.source_in
+
+    @property
+    def source_out(self) -> str:
+        """
+        Get the source out timecode.
+        """
+
+        return self.edit_statement.source_out
+
+    @property
+    def record_in(self) -> str:
+        """
+        Get the record in timecode.
+        """
+
+        return self.edit_statement.record_in
+
+    @property
+    def record_out(self) -> str:
+        """
+        Get the record out timecode.
+        """
+
+        return self.edit_statement.record_out
+
+    @property
+    def source(self) -> str:
+        """
+        Get the source column. This is the 8, 32 or 128-character string on the
+        event record line, this usually references the tape name of the source.
+        """
+        return self.edit_statement.source
+
+    @property
+    def black(self) -> bool:
+        """
+        Black video or silence should be used as the source for this event.
+        """
+        return self.source == "BL"
+
+    @property
+    def aux_source(self) -> bool:
+        """
+        An auxiliary source is the source of this event.
+        """
+        return self.source == "AX"
+
+    @property
+    def source_file(self) -> Optional[str]:
+        """
+        Get the source file, as attested by a "* SOURCE FILE" remark on the
+        EDL. This will return None if the information is not present.
+        """
+        if self.source_file_statement is None:
+            return None
+        else:
+            return self.source_file_statement.filename
+
+    @property
+    def clip_name(self) -> Optional[str]:
+        """
+        Get the clip name, as attested by a "* FROM CLIP NAME" or "* TO CLIP 
+        NAME" remark on the EDL. This will return None if the information is
+        not present.
+        """
+        if self.clip_name_statement is None:
+            return None
+        else:
+            return self.clip_name_statement.name
+
+

pycmx/edit_list.py

@@ -0,0 +1,109 @@
+# pycmx
+# (c) 2018 Jamie Hardt
+
+from .parse_cmx_statements import (StmtUnrecognized, StmtFCM, 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()`.
+    """
+    def __init__(self, statements):
+        self.title_statement = statements[0]
+        self.event_statements = statements[1:]
+
+    @property
+    def format(self) -> str:
+        """
+        The detected format of the EDL. Possible values are: "3600", "File32",
+        "File128", and "unknown".
+        """
+        first_event = next( (s for s in self.event_statements if type(s) is StmtEvent), None)
+
+        if first_event:
+            if first_event.format == 8:
+                return '3600'
+            elif first_event.format == 32:
+                return 'File32'
+            elif first_event.format == 128:
+                return 'File128'
+            else:
+                return 'unknown'
+        else:
+            return 'unknown'
+        
+
+    @property
+    def channels(self) -> ChannelMap:
+        """
+        Return the union of every channel channel.
+        """
+
+        retval = ChannelMap()
+        for event in self.events:
+            for edit in event.edits:
+                retval = retval | edit.channels
+
+        return retval
+        
+
+    @property
+    def title(self) -> str:
+        """
+        The title of this edit list.
+        """
+        return self.title_statement.title
+
+    
+    @property
+    def unrecognized_statements(self) -> Generator[StmtUnrecognized, None, None]:
+        """
+        A generator for all the unrecognized statements in the list.
+        """
+        for s in self.event_statements:
+            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 current_event_num is None:
+                    current_event_num = stmt.event
+                    event_statements.append(stmt)
+                else:
+                    if current_event_num != stmt.event:
+                        yield Event(statements=event_statements)
+                        event_statements = [stmt]
+                        current_event_num = stmt.event
+                    else:
+                        event_statements.append(stmt)
+
+            elif type(stmt) is StmtSourceUMID:
+                break
+            else:
+                event_statements.append(stmt)
+
+        yield Event(statements=event_statements)
+
+    @property
+    def sources(self) -> Generator[StmtSourceUMID, None, None]:
+        """
+        A generator for all of the sources in the list
+        """
+        
+        for stmt in self.event_statements:
+            if type(stmt) is StmtSourceUMID:
+                yield stmt
+
+

pycmx/event.py

@@ -0,0 +1,104 @@
+# pycmx
+# (c) 2023 Jamie Hardt
+
+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.
+    """
+
+    def __init__(self, statements):
+        self.statements = statements
+    
+    @property
+    def number(self) -> int:
+        """
+        Return the event number.
+        """
+        return int(self._edit_statements()[0].event)
+
+    @property
+    def edits(self) -> List[Edit]:
+        """
+        Returns the edits. Most events will have a single edit, a single event
+        will have multiple edits when a dissolve, wipe or key transition needs
+        to be performed.
+        """
+        edits_audio = list( self._statements_with_audio_ext() )
+        clip_names  = self._clip_name_statements()
+        source_files= self._source_file_statements()
+         
+        the_zip: List[List[Any]] = [edits_audio]
+
+        if len(edits_audio) == 2:
+            start_name: Optional[StmtClipName] = None
+            end_name: Optional[StmtClipName] = None
+
+            for clip_name in clip_names:
+                if clip_name.affect == 'from':
+                    start_name = clip_name
+                elif clip_name.affect == 'to':
+                    end_name = clip_name
+
+            the_zip.append([start_name, end_name])
+        else:    
+            if len(edits_audio) == len(clip_names):
+                the_zip.append(clip_names)
+            else:
+                the_zip.append([None] * len(edits_audio) )
+
+        if len(edits_audio) == len(source_files):
+            the_zip.append(source_files)
+        elif len(source_files) == 1:
+            the_zip.append( source_files * len(edits_audio) )
+        else:
+            the_zip.append([None] * len(edits_audio) )
+        
+        # attach trans name to last event
+        try:
+            trans_statement = self._trans_name_statements()[0]
+            trans_names: List[Optional[Any]] = [None] * (len(edits_audio) - 1)
+            trans_names.append(trans_statement)
+            the_zip.append(trans_names)
+        except IndexError:
+            the_zip.append([None] * len(edits_audio) )
+
+        return [ Edit(edit_statement=e1[0],
+                      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) ]
+            
+    @property
+    def unrecognized_statements(self) -> Generator[StmtUnrecognized, None, None]:
+        """
+        A generator for all the unrecognized statements in the event.
+        """
+        for s in self.statements:
+            if type(s) is StmtUnrecognized:
+                yield s
+    
+    def _trans_name_statements(self) -> List[StmtEffectsName]:
+        return [s for s in self.statements if type(s) is StmtEffectsName]
+
+    def _edit_statements(self) -> List[StmtEvent]:
+        return [s for s in self.statements if type(s) is StmtEvent]
+
+    def _clip_name_statements(self) -> List[StmtClipName]:
+        return [s for s in self.statements if type(s) is StmtClipName]
+    
+    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]:
+        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

@@ -1,348 +1,20 @@
 # pycmx
 # (c) 2018 Jamie Hardt
 
-from .parse_cmx_statements import (parse_cmx3600_statements, 
-        StmtEvent, StmtFCM, StmtTitle, StmtClipName, StmtSourceFile, StmtAudioExt, StmtUnrecognized)
+# from collections import namedtuple
 
-from .channel_map import ChannelMap
+from .parse_cmx_statements import (parse_cmx3600_statements)
+from .edit_list import EditList
 
-from collections import namedtuple
+from typing import TextIO 
 
-def parse_cmx3600(f):
+def parse_cmx3600(f: TextIO):
     """
     Parse a CMX 3600 EDL.
 
-    Args:
-        f : a file-like object, anything that's readlines-able.
-
-    Returns:
-        An :obj:`EditList`.
+    :param TextIO f: a file-like object, anything that's readlines-able.
+    :returns: An :class:`pycmx.edit_list.EditList`.
     """
     statements = parse_cmx3600_statements(f)
     return EditList(statements)
     
-    
-class EditList:
-    """
-    Represents an entire edit decision list as returned by `parse_cmx3600()`.
-    
-    """
-    def __init__(self, statements):
-        self.title_statement = statements[0]
-        self.event_statements = statements[1:]
-
-    @property
-    def title(self):
-        """
-        The title of this edit list, as attensted by the 'TITLE:' statement on 
-        the first line.
-        """
-        'The title of the edit list'
-        return self.title_statement.title
-
-    
-    @property
-    def unrecognized_statements(self):
-        """
-        A generator for all the unrecognized statements in the list.
-        """
-        for s in self.event_statements:
-            if type(s) is StmtUnrecognized:
-                yield s
-        
-
-    @property
-    def events(self):
-        '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 current_event_num is None:
-                    current_event_num = stmt.event
-                    event_statements.append(stmt)
-                else:
-                    if current_event_num != stmt.event:
-                        yield Event(statements=event_statements)
-                        event_statements = [stmt]
-                        current_event_num = stmt.event
-                    else:
-                        event_statements.append(stmt)
-
-            else:
-                event_statements.append(stmt)
-
-        yield Event(statements=event_statements)
-
-
-class Edit:
-    def __init__(self, edit_statement, audio_ext_statement, clip_name_statement, source_file_statement, other_statements = []):
-        self.edit_statement = edit_statement
-        self.audio_ext = audio_ext_statement
-        self.clip_name_statement = clip_name_statement
-        self.source_file_statement = source_file_statement
-        self.other_statements = other_statements
-
-    @property
-    def line_number(self):
-        """
-        Get the line number for the "standard form" statement associated with
-        this edit. Line numbers a zero-indexed, such that the 
-        "TITLE:" record is line zero.
-        """
-        return self.edit_statement.line_number
-
-    @property
-    def channels(self):
-        """
-        Get the :obj:`ChannelMap` object associated with this Edit.
-        """
-        cm = ChannelMap()
-        cm._append_event(self.edit_statement.channels)
-        if self.audio_ext != None:
-            cm._append_ext(self.audio_ext)
-        return cm
-
-    @property
-    def transition(self):
-        """
-        Get the :obj:`Transition` object associated with this edit.
-        """
-        return Transition(self.edit_statement.trans, self.edit_statement.trans_op)
-   
-    @property
-    def source_in(self):
-        """
-        Get the source in timecode.
-        """
-        return self.edit_statement.source_in
-
-    @property
-    def source_out(self):
-        """
-        Get the source out timecode.
-        """
-
-        return self.edit_statement.source_out
-
-    @property
-    def record_in(self):
-        """
-        Get the record in timecode.
-        """
-
-        return self.edit_statement.record_in
-
-    @property
-    def record_out(self):
-        """
-        Get the record out timecode.
-        """
-
-        return self.edit_statement.record_out
-
-    @property
-    def source(self):
-        """
-        Get the source column. This is the 8, 32 or 128-character string on the
-        event record line, this usually references the tape name of the source.
-        """
-        return self.edit_statement.source
-
-
-    @property
-    def source_file(self):
-        """
-        Get the source file, as attested by a "* SOURCE FILE" remark on the
-        EDL. This will return None if the information is not present.
-        """
-        if self.source_file_statement is None:
-            return None
-        else:
-            return self.source_file_statement.filename
-
-
-    @property
-    def clip_name(self):
-        """
-        Get the clip name, as attested by a "* FROM CLIP NAME" or "* TO CLIP 
-        NAME" remark on the EDL. This will return None if the information is
-        not present.
-        """
-        if self.clip_name_statement != None:
-            return self.clip_name_statement.name
-        else:
-            return None
-        
-
-
-class Event:
-    """
-    Represents a collection of :obj:`Edit`s, all with the same event number.
-    """
-
-    def __init__(self, statements):
-        self.statements = statements
-    
-    @property
-    def number(self):
-        """Return the event number."""
-        return int(self._edit_statements()[0].event)
-
-    @property
-    def edits(self):
-        """
-        Returns the edits. Most events will have a single edit, a single event
-        will have multiple edits when a dissolve, wipe or key transition needs
-        to be performed.
-        """
-        edits_audio = list( self._statements_with_audio_ext() )
-        clip_names  = self._clip_name_statements()
-        source_files= self._source_file_statements()
-        
-        the_zip = [edits_audio]
-
-        if len(edits_audio) == 2:
-            cn = [None, None]
-            for clip_name in clip_names:
-                if clip_name.affect == 'from':
-                    cn[0] = clip_name
-                elif clip_name.affect == 'to':
-                    cn[1] = clip_name
-
-            the_zip.append(cn)
-
-        else:    
-            if len(edits_audio) == len(clip_names):
-                the_zip.append(clip_names)
-            else:
-                the_zip.append([None] * len(edits_audio) )
-
-        if len(edits_audio) == len(source_files):
-            the_zip.append(source_files)
-        elif len(source_files) == 1:
-            the_zip.append( source_files * len(edits_audio) )
-        else:
-            the_zip.append([None] * len(edits_audio) )
-
-
-        return [ Edit(e1[0],e1[1],n1,s1) for (e1,n1,s1) in zip(*the_zip) ]
-            
-    @property
-    def unrecognized_statements(self):
-        """
-        A generator for all the unrecognized statements in the event.
-        """
-        for s in self.statements:
-            if type(s) is StmtUnrecognized:
-                yield s            
-
-    def _edit_statements(self):
-        return [s for s in self.statements if type(s) is StmtEvent]
-
-    def _clip_name_statements(self):
-        return [s for s in self.statements if type(s) is StmtClipName]
-    
-    def _source_file_statements(self):
-        return [s for s in self.statements if type(s) is StmtSourceFile]
-    
-    def _statements_with_audio_ext(self):
-        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)
-
-    
-
-class Transition:
-    """Represents a CMX transition, a wipe, dissolve or cut."""
-
-    Cut = "C"
-
-    Dissolve = "D"
-
-    Wipe = "W"
-
-    KeyBackground = "KB"
-
-    Key = "K"
-
-    KeyOut = "KO"
-
-
-    def __init__(self, transition, operand):
-        self.transition = transition
-        self.operand = operand
-        self.name = ''
-
-
-    @property
-    def kind(self):
-        """
-        Return the kind of transition: Cut, Wipe, etc
-        """
-        if self.cut:
-            return Transition.Cut
-        elif self.dissolve:
-            return Transition.Dissolve
-        elif self.wipe:
-            return Transition.Wipe
-        elif self.key_background:
-            return Transition.KeyBackground
-        elif self.key_foreground:
-            return Transition.Key
-        elif self.key_out:
-            return Transition.KeyOut
-
-    @property
-    def cut(self):
-        "`True` if this transition is a cut."
-        return self.transition == 'C' 
-
-    @property
-    def dissolve(self):
-        "`True` if this traansition is a dissolve."
-        return self.transition == 'D'
-
-
-    @property
-    def wipe(self):
-        "`True` if this transition is a wipe."
-        return self.transition.startswith('W')
-
-
-    @property
-    def effect_duration(self):
-        """The duration of this transition, in frames of the record target.
-        
-        In the event of a key event, this is the duration of the fade in.
-        """
-        return int(self.operand)
-
-    @property
-    def wipe_number(self):
-        "Wipes are identified by a particular number."
-        if self.wipe:
-            return int(self.transition[1:])
-        else:
-            return None
-
-    @property
-    def key_background(self):
-        "`True` if this is a key background event."
-        return self.transition == KeyBackground
-
-    @property
-    def key_foreground(self):
-        "`True` if this is a key foreground event."
-        return self.transition == Key
-
-    @property
-    def key_out(self):
-        "`True` if this is a key out event."
-        return self.transition == KeyOut
-

pycmx/parse_cmx_statements.py

@@ -1,29 +1,31 @@
 # pycmx
 # (c) 2018 Jamie Hardt
 
-from .util import collimate
 import re
 import sys
 from collections import namedtuple
 from itertools import count
+from typing import TextIO, List
 
 
+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","line_number"])
+        "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"])
 StmtRemark =    namedtuple("Remark",["text","line_number"])
 StmtEffectsName = namedtuple("EffectsName",["name","line_number"])
-StmtTrailer =   namedtuple("Trailer",["text","line_number"])
-StmtSplitEdit = namedtuple("SplitEdit",["video","magnitue", "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
 StmtUnrecognized = namedtuple("Unrecognized",["content","line_number"])
 
 
-def parse_cmx3600_statements(file):
+def parse_cmx3600_statements(file: TextIO) -> List[object]:
     """
     Return a list of every statement in the file argument.
     """
@@ -69,8 +71,8 @@ def _parse_cmx3600_line(line, line_number):
             return _parse_extended_audio_channels(line,line_number)
         elif line.startswith("*"):
             return _parse_remark( line[1:].strip(), line_number)
-        elif line.startswith(">>>"):
-            return _parse_trailer_statement(line, line_number)
+        elif line.startswith(">>> SOURCE"):
+            return _parse_source_umid_statement(line, line_number)
         elif line.startswith("EFFECTS NAME IS"):
             return _parse_effects_name(line, line_number)
         elif line.startswith("SPLIT:"):
@@ -109,7 +111,7 @@ def _parse_extended_audio_channels(line, line_number):
     else:
         return StmtUnrecognized(content=line, line_number=line_number)
     
-def _parse_remark(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)
     elif line.startswith("TO CLIP NAME:"):
@@ -119,7 +121,7 @@ def _parse_remark(line, line_number):
     else:
         return StmtRemark(text=line, line_number=line_number)
 
-def _parse_effects_name(line, line_number):
+def _parse_effects_name(line, line_number) -> StmtEffectsName:
     name = line[16:].strip()
     return StmtEffectsName(name=name, line_number=line_number)
 
@@ -157,10 +159,11 @@ 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)
+		    line_number=line_number,
+                    format=source_field_length)
 
 
-def _parse_trailer_statement(line, line_number):
+def _parse_source_umid_statement(line, line_number):
     trimmed = line[3:].strip()
-    return StmtTrailer(trimmed, line_number=line_number)
+    return StmtSourceUMID(name=None, umid=None, line_number=line_number)
 

pycmx/transition.py

@@ -0,0 +1,90 @@
+# pycmx
+# (c) 2023 Jamie Hardt
+
+from typing import Optional
+
+class Transition:
+    """
+    A CMX transition: a wipe, dissolve or cut.
+    """
+    
+    Cut = "C"
+    Dissolve = "D"
+    Wipe = "W"
+    KeyBackground = "KB"
+    Key = "K"
+    KeyOut = "KO"
+
+    def __init__(self, transition, operand, name=None):
+        self.transition = transition
+        self.operand = operand
+        self.name = name
+
+    @property
+    def kind(self) -> Optional[str]:
+        """
+        Return the kind of transition: Cut, Wipe, etc
+        """
+        if self.cut:
+            return Transition.Cut
+        elif self.dissolve:
+            return Transition.Dissolve
+        elif self.wipe:
+            return Transition.Wipe
+        elif self.key_background:
+            return Transition.KeyBackground
+        elif self.key_foreground:
+            return Transition.Key
+        elif self.key_out:
+            return Transition.KeyOut
+
+    @property
+    def cut(self) -> bool:
+        "`True` if this transition is a cut."
+        return self.transition == 'C' 
+
+    @property
+    def dissolve(self) -> bool:
+        "`True` if this traansition is a dissolve."
+        return self.transition == 'D'
+
+    @property
+    def wipe(self) -> bool:
+        "`True` if this transition is a wipe."
+        return self.transition.startswith('W')
+
+    @property
+    def effect_duration(self) -> int:
+        """The duration of this transition, in frames of the record target.
+        
+        In the event of a key event, this is the duration of the fade in.
+        """
+        return int(self.operand)
+
+    @property
+    def wipe_number(self) -> Optional[int]:
+        "Wipes are identified by a particular number."
+        if self.wipe:
+            return int(self.transition[1:])
+        else:
+            return None
+
+    @property
+    def key_background(self) -> bool:
+        "`True` if this edit is a key background."
+        return self.transition == Transition.KeyBackground
+
+    @property
+    def key_foreground(self) -> bool:
+        "`True` if this edit is a key foreground."
+        return self.transition == Transition.Key
+
+    @property
+    def key_out(self) -> bool:
+        """
+        `True` if this edit is a key out. This material will removed from
+        the key foreground and replaced with the key background.
+        """
+        return self.transition == Transition.KeyOut
+
+

pyproject.toml

@@ -0,0 +1,56 @@
+[tool.poetry]
+name = "pycmx"
+version = "1.2.3"
+description = "Python CMX 3600 Edit Decision List Parser"
+authors = ["Jamie Hardt <jamiehardt@me.com>"]
+license = "MIT"
+readme = "README.md"
+keywords = [
+  'parser',
+  'film', 
+  'broadcast' 
+]
+classifiers = [
+  'Development Status :: 5 - Production/Stable',
+  'License :: OSI Approved :: MIT License',
+  'Topic :: Multimedia',
+  'Topic :: Multimedia :: Video',
+  'Topic :: Text Processing',
+  'Programming Language :: Python :: 3.8',
+  'Programming Language :: Python :: 3.9',
+  'Programming Language :: Python :: 3.10',
+  'Programming Language :: Python :: 3.11',
+  'Programming Language :: Python :: 3.12',
+  'Programming Language :: Python :: 3.13'
+]
+homepage = "https://github.com/iluvcapra/pycmx"
+documentation = "https://pycmx.readthedocs.io/"
+repository = "https://github.com/iluvcapra/pycmx.git"
+urls.Tracker = "https://github.com/iluvcapra/pycmx/issues" 
+
+[tool.poetry.extras]
+doc = ['sphinx', 'sphinx_rtd_theme']
+
+[tool.poetry.dependencies]
+python = "^3.8"
+sphinx = { version='>= 5.3.0', optional=true}
+sphinx_rtd_theme = {version ='>= 1.1.1', optional=true}
+
+[tool.pyright]
+typeCheckingMode = "basic"
+
+[tool.pylint]
+max-line-length = 88
+disable = [
+    "C0103", # (invalid-name)
+    "C0114", # (missing-module-docstring)
+    "C0115", # (missing-class-docstring)
+    "C0116", # (missing-function-docstring)
+    "R0903", # (too-few-public-methods)
+    "R0913", # (too-many-arguments)
+    "W0105", # (pointless-string-statement)
+]
+
+[build-system]
+requires = ["poetry-core"]
+build-backend = "poetry.core.masonry.api"

setup.py

@@ -1,19 +0,0 @@
-from setuptools import setup
-
-with open("README.md", "r") as fh:
-    long_description = fh.read()
-
-setup(name='pycmx',
-      version='0.7',
-      author='Jamie Hardt',
-      author_email='jamiehardt@me.com',
-      description='CMX 3600 Edit Decision List Parser',
-      long_description_content_type="text/markdown",
-      long_description=long_description,
-      url='https://github.com/iluvcapra/pycmx',
-      classifiers=['Development Status :: 4 - Beta',
-          'License :: OSI Approved :: MIT License',
-          'Topic :: Multimedia',
-          'Topic :: Multimedia :: Video',
-          'Topic :: Text Processing'],
-      packages=['pycmx'])

tests/__init__.py

@@ -0,0 +1 @@
+from . import test_parse

tests/test_parse.py

@@ -18,21 +18,24 @@ class TestParse(TestCase):
         counts = [ 287, 466, 250 , 376, 120 , 3 , 466 ]
 
         for fn, count in zip(type(self).files, counts):
-            with open(f"tests/edls/{fn}" ,'r') as f:
+            with open("tests/edls/" + fn ,'r') as f:
                 edl = pycmx.parse_cmx3600(f)
                 actual = len( list( edl.events ))
-                self.assertTrue( actual == count , f"expected {count} in file {fn} but found {actual}")
+                self.assertTrue( actual == count ,
+                        "expected %i in file %s but found %i" % (count, fn, actual))
 
     def test_list_sanity(self):
         for fn in type(self).files:
-            with open(f"tests/edls/{fn}" ,'r') as f:
+            with open("tests/edls/" + fn ,'r') as f:
                 edl = pycmx.parse_cmx3600(f)
                 self.assertTrue( type(edl.title) is str )
-    
+                self.assertTrue( len(edl.title) > 0 )
+
 
     def test_event_sanity(self):
         for fn in type(self).files:
-            with open(f"tests/edls/{fn}" ,'r') as f:
+            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 )
@@ -44,7 +47,7 @@ class TestParse(TestCase):
         with open("tests/edls/TEST.edl",'r') as f:
             edl = pycmx.parse_cmx3600(f)
             events = list( edl.events  )
-        
+
             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")
@@ -63,6 +66,7 @@ class TestParse(TestCase):
             self.assertFalse( events[0].edits[0].channels.a1)
             self.assertTrue( events[0].edits[0].channels.a2)
             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):
@@ -98,4 +102,12 @@ class TestParse(TestCase):
             self.assertEqual( events[14].edits[0].line_number, 45)
             self.assertEqual( events[180].edits[0].line_number, 544)
 
+    def test_transition_name(self):
+        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" )
+
+
+    # add test for edit_list.channels