Page MenuHome

Video SequencerProject
ActivePublic

Details

Description

Recent Activity

Wed, Jun 19

Richard Antalik (ISS) closed T65398: Strip info doesn't use custom frame rate base as Resolved by committing rBda83f70754b6: Fix T65398 - Fix frame rate base use in Python SMPTE functions.
Wed, Jun 19, 7:53 PM · Video Sequencer, BF Blender

Mon, Jun 17

Peter Fog (tintwotin) added a comment to T65845: VSE remaining UI inconsistencies.

@Richard Antalik (ISS) - If you have a moment while going through the VSE functions when updating the manual, could you check the values in the "Timecode" Panel, if they make sense to you? For me, several of them do not. Here are my notes:

  • Is it correct 'frame_start' should show the mouse position in relation to original insertion point(or something like that)? - this value is no longer used in the panel.
  • 'frame_end' seems broken(not used in panel).
  • If the Hold Offset Start is less than the start point of Strip Offset Start, and Strip Offset Start value is changes it will jump to zero, and if extended to the left the hold part of the strip will not be drawn, until the Hold Offset Start value is changed.
  • The 'Strip Offset End' shows the number of frames to the end of the source material, this is not very useful in the UI. Ex. to get the strip end to extend to the right(make it longer) you'll have to drag to the left(opposite direction) to decrease the value.
  • The Hold Offset Start/animation_offset_start seems completely off. It extends the strip in the end, not in the start.
  • There are commented out hold frame duration values in the code.
  • Several of the values can be illegal, causing the strips to turn the 'inside out".
  • The panel titel "Timecodes" can be misleading - because often time codes means in most cases file embedded time codes. This panel is just Transform values written in SMPTE and frames - and not time codes.
Mon, Jun 17, 5:29 PM · Video Sequencer
Sebastian Parborg (zeddb) assigned T65723: Sequencer: Invalid timecode values makes strip drawing a mess. to Richard Antalik (ISS).
Mon, Jun 17, 2:27 PM · Video Sequencer, BF Blender
Brecht Van Lommel (brecht) added a comment to T65845: VSE remaining UI inconsistencies.

We are hiding entries in strip menu based on selection. Should they be preferably grayed?

Mon, Jun 17, 1:25 PM · Video Sequencer
Richard Antalik (ISS) updated the task description for T65845: VSE remaining UI inconsistencies.
Mon, Jun 17, 11:37 AM · Video Sequencer
Samuel Nicholas (enetheru) added a comment to T65845: VSE remaining UI inconsistencies.

Sorry if this is the wrong place to write this, but in the markers menu->jump to next marker, previous marker should instead be in the view->navigation menu instead with the other jump to commands?

Mon, Jun 17, 2:59 AM · Video Sequencer
Richard Antalik (ISS) updated the task description for T65845: VSE remaining UI inconsistencies.
Mon, Jun 17, 12:41 AM · Video Sequencer
Richard Antalik (ISS) updated the task description for T65845: VSE remaining UI inconsistencies.
Mon, Jun 17, 12:29 AM · Video Sequencer

Sun, Jun 16

Richard Antalik (ISS) updated the task description for T65845: VSE remaining UI inconsistencies.
Sun, Jun 16, 4:23 PM · Video Sequencer
Richard Antalik (ISS) updated the task description for T65845: VSE remaining UI inconsistencies.
Sun, Jun 16, 4:00 PM · Video Sequencer
Richard Antalik (ISS) updated subscribers of T65845: VSE remaining UI inconsistencies.

If the hidden elements(many different strip types and their options) all should be visible, but greyed out, it would be overkill. As it is now with a submenu changing contents depending on the current active strip will mean less clutter IMO, however image and sound strips does not have a sub-menu, because there are too few entries.

Sun, Jun 16, 3:58 PM · Video Sequencer
Peter Fog (tintwotin) added a comment to T65845: VSE remaining UI inconsistencies.

The contents of SEQUENCER_MT_change seems to have been moved into the Input menu in this commit: D2665

Sun, Jun 16, 2:39 PM · Video Sequencer
Richard Antalik (ISS) triaged T65845: VSE remaining UI inconsistencies as Normal priority.
Sun, Jun 16, 1:16 AM · Video Sequencer

Sat, Jun 15

Peter Fog (tintwotin) added a comment to T65829: The width of the VSE Sidebar has become too narrow .

This is the width of the VSE sidebar panel from a build I downloaded 19/5 called blender-2.80.0-git.06c4139a6833-windows64:

Sat, Jun 15, 1:25 PM · Video Sequencer, User Interface, BF Blender
William Reynish (billreynish) claimed T65828: VSE Cache Sidebar panel is not right aligned.
Sat, Jun 15, 12:14 PM · Video Sequencer, User Interface, BF Blender
William Reynish (billreynish) closed T65829: The width of the VSE Sidebar has become too narrow as Invalid.

This is not a bug, and not something that can be addressed with the system we have. The width of sidebars are constant, while the size of editors are relative to the window size. This means that the they will differ depending on the window size - at certain sizes they *will* be identical in width.

Sat, Jun 15, 11:52 AM · Video Sequencer, User Interface, BF Blender
Peter Fog (tintwotin) added projects to T65828: VSE Cache Sidebar panel is not right aligned: User Interface, Video Sequencer.
Sat, Jun 15, 9:05 AM · Video Sequencer, User Interface, BF Blender
Peter Fog (tintwotin) added projects to T65829: The width of the VSE Sidebar has become too narrow : User Interface, Video Sequencer.
Sat, Jun 15, 9:05 AM · Video Sequencer, User Interface, BF Blender
Peter Fog (tintwotin) abandoned D5046: Set Timecode Start value to match Timecode End and Duration values.

This has been fixed by D5065.

Sat, Jun 15, 8:42 AM · Video Sequencer, BF Blender

Thu, Jun 13

Peter Fog (tintwotin) added a project to T65746: VSE: Missing import settings in the workspace file browser can cause slow playback and a/v duration mismatch: User Interface.
Thu, Jun 13, 11:35 PM · User Interface, Video Sequencer, BF Blender
William Reynish (billreynish) closed D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Thu, Jun 13, 3:54 PM · Video Sequencer, BF Blender
William Reynish (billreynish) accepted D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Thu, Jun 13, 3:42 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) updated the summary of D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Thu, Jun 13, 11:53 AM · Video Sequencer, BF Blender
Samuel Nicholas (enetheru) added a watcher for Video Sequencer: Samuel Nicholas (enetheru).
Thu, Jun 13, 8:36 AM
Nathan Lovato (gdquest) added a comment to T65724: VSE operators rework.

user has to save selection state, do stuff, and restore selection. Not a good practice IMO.

Thu, Jun 13, 2:26 AM · Video Sequencer
Campbell Barton (campbellbarton) added a comment to T65724: VSE operators rework.

Although these two proposals would need discussion and approval in separate tasks, what I got from the discussion with Richard is that the point of this task is to discuss and get approval to have more Python code for VSE operators, to make it a lot easier for add-on and Python devs like me to contribute.
Are new tools or ops written in Python welcome in the Sequencer project?

Thu, Jun 13, 12:25 AM · Video Sequencer
Campbell Barton (campbellbarton) added a comment to T65724: VSE operators rework.
  • Moving images_separate to an API call then adding the operator from Python seems like a token. All the Python operator will do is call the C API code which will have all the logic. How is this an improvement?

Operator will not return reference to created strips. It can do this only indirectly by selecting them, them.
By doing this user has to save selection state, do stuff, and restore selection. Not a good practice IMO.

Thu, Jun 13, 12:10 AM · Video Sequencer

Wed, Jun 12

Richard Antalik (ISS) added a comment to T65724: VSE operators rework.
  • Moving images_separate to an API call then adding the operator from Python seems like a token. All the Python operator will do is call the C API code which will have all the logic. How is this an improvement?
Wed, Jun 12, 7:45 PM · Video Sequencer
Richard Antalik (ISS) updated the task description for T65724: VSE operators rework.
Wed, Jun 12, 7:40 PM · Video Sequencer
Nathan Lovato (gdquest) added a comment to T65724: VSE operators rework.

Richard asked for some new operators or features we'd like to suggest as part of this project. I wrote and played quite a bit with the VSE with our add-on power sequencer. I've written operators that help to edit all kinds of content faster than with vanilla blender:

Wed, Jun 12, 4:44 PM · Video Sequencer
Nathan Lovato (gdquest) added a comment to T65724: VSE operators rework.

The idea was to make it possible for Python developers to contribute to the VSE more easily and maintain these operators. The operators being object-like, they're easier to write, read, and encapsulate in Python: in C you need several functions.

Wed, Jun 12, 4:22 PM · Video Sequencer
Peter Fog (tintwotin) added a comment to T65746: VSE: Missing import settings in the workspace file browser can cause slow playback and a/v duration mismatch.

Pre 2.80 users could only import Movies through the Add menu and therefore they where always was exposed to the import options. With the file browser in the default VSE workspace missing the import settings, most people will never know the existence of them, causing a bad experience.

Wed, Jun 12, 11:36 AM · User Interface, Video Sequencer, BF Blender
Brecht Van Lommel (brecht) closed T65746: VSE: Missing import settings in the workspace file browser can cause slow playback and a/v duration mismatch as Invalid.

This is a usability issue, outside the scope of the bug tracker.

Wed, Jun 12, 11:30 AM · User Interface, Video Sequencer, BF Blender
Peter Fog (tintwotin) added a project to T65746: VSE: Missing import settings in the workspace file browser can cause slow playback and a/v duration mismatch: Video Sequencer.
Wed, Jun 12, 11:12 AM · User Interface, Video Sequencer, BF Blender
Campbell Barton (campbellbarton) added a comment to T65724: VSE operators rework.

Proposals like this should explain the motivation (pros & cons).
What advantage does this have to rewrite these in another language?

Wed, Jun 12, 7:53 AM · Video Sequencer
Peter Fog (tintwotin) updated the summary of D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Wed, Jun 12, 12:27 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a comment to D5046: Set Timecode Start value to match Timecode End and Duration values.

This patch has been included in this more comprehensive timecode panel fix patch: D5065

Wed, Jun 12, 12:17 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) updated the summary of D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Wed, Jun 12, 12:10 AM · Video Sequencer, BF Blender
Peter Fog (tintwotin) created D5065: Sequencer Timecode Panel - fix invalid values, fix alignment and fix renaming.
Wed, Jun 12, 12:04 AM · Video Sequencer, BF Blender

Tue, Jun 11

Christopher Anderssarian (Christopher_Anderssarian) added a comment to T65398: Strip info doesn't use custom frame rate base.

I would agree with :FFF format for frame regardless of framerate.

Not what I had in mind (but good for a quick fix). That placeholder wouldn't be needed 99% of the time.

Tue, Jun 11, 11:40 PM · Video Sequencer, BF Blender
Richard Antalik (ISS) added a revision to T65398: Strip info doesn't use custom frame rate base: D5064: Fix T65398 - Python SMPTE functions doesn't use custom frame rate base correctly.
Tue, Jun 11, 11:22 PM · Video Sequencer, BF Blender
Richard Antalik (ISS) added a comment to T65398: Strip info doesn't use custom frame rate base.

@Christopher Anderssarian (Christopher_Anderssarian)
I would agree with :FFF format for frame regardless of framerate. Each time I look at strip length now, I have to think about the format. But according to https://en.wikipedia.org/wiki/SMPTE_timecode, the format is HH:MM:SS:FF, which we are violating already.

Tue, Jun 11, 11:18 PM · Video Sequencer, BF Blender
Richard Antalik (ISS) added a comment to T65720: Fast image processing library.

@Jeroen Bakker (jbakker)
Thanks for reminding me of halide - forgot about that one.

Tue, Jun 11, 6:17 PM · Video Sequencer
Jeroen Bakker (jbakker) added a comment to T65720: Fast image processing library.

What are the requirements of such a library. I would expect that headless rendering is one of those.
Did you have a look at Halide? https://halide-lang.org/
They promote them as a language, but actually is more a framework.

Tue, Jun 11, 4:48 PM · Video Sequencer
Peter Fog (tintwotin) added a comment to T65723: Sequencer: Invalid timecode values makes strip drawing a mess..

Just checked 2.73. Same bug.

Tue, Jun 11, 4:42 PM · Video Sequencer, BF Blender
Richard Antalik (ISS) updated the task description for T64682: Video Sequence Editor (VSE) Module.
Tue, Jun 11, 3:31 PM · Video Sequencer, BF Blender
Richard Antalik (ISS) triaged T65724: VSE operators rework as Normal priority.
Tue, Jun 11, 3:30 PM · Video Sequencer
Richard Antalik (ISS) created T65724: VSE operators rework.
Tue, Jun 11, 3:29 PM · Video Sequencer
Richard Antalik (ISS) added a comment to T65723: Sequencer: Invalid timecode values makes strip drawing a mess..

Is this new issue?
I mean it should be easy to fix, but not sure if I should look for what caused this.

Tue, Jun 11, 3:18 PM · Video Sequencer, BF Blender
Peter Fog (tintwotin) added a project to T65723: Sequencer: Invalid timecode values makes strip drawing a mess.: Video Sequencer.
Tue, Jun 11, 3:13 PM · Video Sequencer, BF Blender