Done!

In T68584#769812, @Nathan Lovato (gdquest) wrote:

• Is there anything left to clean up, or can I delete the temp branch and push the addon?
• If so, can I push the addon straight to the blender-addons repository?

Got it, makes total sense. You tend to forget how things work at a lower level working with Python, at least until you hit performance issues. So even if poll gets context as an argument, until you access a given list or collection, it isn't even initialized, right. I'll be mindful of that when contributing to the VSE.

Re: checking lengths:

Thanks, I'm aware of that and wouldn't do it in the official source code. This looks like something I did a long time ago, I'll clean it up.

Went over the code in some more detail, generally seems OK.

It's up in the addons-contrib repo, branch temp-power-sequencer, I added a power_sequencer directory: https://developer.blender.org/diffusion/BAC/browse/temp-power-sequencer/

Sure, I'll do that after work! Thanks

@Nathan Lovato (gdquest) could you commit to a temporary branch? tracking changes between updates to a zip-file doesn't work well, it's inconvenient to see what's changed.

I updated my build script to clean up all the extra files, added the license headers, and made all the changes:

This is a fairly large code-base, here are some issues I've run into.

Hi, I updated the add-on above, here's a copy:

hi @Nathan Lovato (gdquest)
Welcome and thanks!
All seems well here apart from the mentioned updater, if you can remove the updater and link in the new file before any commits. We can catch up in blender.chat to finalize any issues.
Opening task until fully resolved and committed in case there are any other issues discovered before the commit.

This add-on bundles it's own updater, this raises the policy issue of allowing add-ons to include online-updating functionality that points to 3rd party repositories.

Exact Steps:

• Open provided file
• Window -> New Window
• Play Animation (spacebar)
• Close Window

Audio is still still playing until end