dump_markers.py returns incorrect timecode for audio markers

[mjiggidy]

From what I can tell, marker timecodes from video tracks are correctly given as sequence timecode where they occur.

From audio tracks, however, marker timecodes appear to be the offset from the beginning of the subclip they live on in the sequence, + sequence start timecode.

Apologies for the crude screenshots here, but here's an example:

This marker should be reported with timecode 01:10:28:09. But instead...

...it is listed as 01:00:01:11, which is the relative offset from the beginning of the clip in the timeline (01:11) + the timecode start of the sequence (01:00:00:00)

[mjiggidy]

Ahah! I think I tracked it down -- the issue seems to be specifically with audio tracks which have RTAS effects applied. In this case, the audio track's component becomes an avb.trackgroups.TrackEffect track group rather than a typical avb.components.Sequence component. So instead of passing it directly into find_track_markers(), we first need to iterate over each of the trackgroup's tracks and pass those to find_track_markers().

I have it working for me, I believe, and I'll plan to submit a pull request once I test more thoroughly. But if anyone else has been struggling with this, hopefully my findings help.

[mjiggidy]

Submitted pull request #32 to address this. Most straightforward way I could find to do this was to pull the "get components from track" logic into its own function, so that it could be recursed into if the track component is a TrackGroup. Happy to answer any questions or address any potential issues!