Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve transition management #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mmartinez1972
Copy link

  • adds ToNameMatcher for identify destination name (TO CLIP NAME) in a transition event.
    While (TO FROM NAME) Name is pushed to the previous entry on the list.
  • Update regular expression for EFFECTS Matcher.
    As Working with Toon Boom Storyboard Pro EDL files, its output uses a different phrasing for EFFECTS comment.
  • Move Location for next_event assignment in EventMatcher.apply().
    Reason:
    After a transition event, the event didn't have value for next_event so it was preventing iteration. Only Cut assigned the property.
    (Tested for my known cases)

* adds ToNameMatcher for identify destination
  name (TO CLIP NAME) in a transition event.
  While (TO FROM NAME) Name is pushed to the
  previous entry on the list.
* Update regular expression for EFFECTS Matcher.
  As Working with Toon Boom Storyboard Pro
  EDL files, its output uses a different
  phrasing for EFFECTS comment.
* Move Location for next_event assignment
  in EventMatcher.apply().
  Reason:
  After a transition event, the event didn't
  have value for next_event so it was preventing
  iteration. Only Cut assigned the property.
  (Tested for my known cases)
@mmartinez1972 mmartinez1972 changed the title improve transition management Improve transition management Feb 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants