Page MenuHome
Differential D10661

Feature: Python API create empty meta sequence for easier nested editing creation
ClosedPublic
Actions

Authored by Félix (Miadim) on Mar 9 2021, 11:44 AM.

Details

Reviewers
Richard Antalik (ISS)
Commits
rBbfad8deb0be0: VSE: Add new_meta RNA API function
Summary

You can test it by doing in python console:

frame_start = 10
m = C.scene.sequence_editor.new_meta("test", 2, frame_start)
m.frame_start = frame_start
v = m.sequences.new_movie("mov", "path", 3, frame_start)
m.frame_final_duration = v.frame_duration

Diff Detail

Repository
rB Blender
Branch
ft_meta_api (branched from master)
Build Status
Buildable 13592
Build 13592: arc lint + arc unit

Event Timeline

Félix (Miadim) requested review of this revision.Mar 9 2021, 11:44 AM
Félix (Miadim) created this revision.
Harbormaster completed remote builds in B13355: Diff 34935.Mar 9 2021, 11:44 AM
Félix (Miadim) edited the summary of this revision. (Show Details)Mar 9 2021, 11:52 AM
Félix (Miadim) edited the summary of this revision. (Show Details)
Félix (Miadim) edited the summary of this revision. (Show Details)
Félix (Miadim) added a comment.Mar 17 2021, 12:38 PM

Any input about this patch?

Richard Antalik (ISS) added a comment.Mar 17 2021, 12:39 PM
In D10661#272651, @Félix (Miadim) wrote:

Any input about this patch?

Yes going to look now

Richard Antalik (ISS) added inline comments.Mar 17 2021, 1:02 PM
source/blender/sequencer/intern/strip_add.c
435

Please hard-code some non 0 length here as well.

This can length can be moved to SeqLoadData later. Or you can change SeqLoadData now so there is end_frame as for other strips (effect, image).

438–439

Because this function is called from python it should not alter selection state. It should only make strip and make sure it is good to use.

Richard Antalik (ISS) requested changes to this revision.Mar 17 2021, 1:03 PM
This revision now requires changes to proceed.Mar 17 2021, 1:03 PM
Félix (Miadim) updated this revision to Diff 35314.Mar 17 2021, 7:02 PM
  • Changes: created seq activation removed. created meta len 1 by default
Harbormaster completed remote builds in B13575: Diff 35314.Mar 17 2021, 7:02 PM
Félix (Miadim) marked 2 inline comments as done.Mar 17 2021, 7:09 PM

I think I did it as expected, tell me, it's my first patch :)

Richard Antalik (ISS) added a comment.Mar 17 2021, 9:01 PM

I overlooked one more issue - comment style see https://wiki.blender.org/wiki/Style_Guide/C_Cpp#Comments

Otherwise it seems to work fine. there seems to bug that prevents resizing meta properly, but I don't think that's caused by this patch, will look into that

Félix (Miadim) added inline comments.Mar 18 2021, 2:45 PM
source/blender/sequencer/intern/strip_add.c
426

@Richard Antalik (ISS) Shall we keep the Main *bmain even if we don't use it?

Félix (Miadim) updated this revision to Diff 35342.Mar 18 2021, 2:51 PM
Félix (Miadim) marked an inline comment as not done.
  • Cleaning: Changed comments syntax
Harbormaster completed remote builds in B13592: Diff 35342.Mar 18 2021, 2:51 PM
Richard Antalik (ISS) accepted this revision.Mar 18 2021, 2:56 PM
This revision is now accepted and ready to land.Mar 18 2021, 2:56 PM
Richard Antalik (ISS) added inline comments.Mar 18 2021, 4:31 PM
source/blender/sequencer/intern/strip_add.c
426

Ah this I haven't noticed as well, no unused arguments would generate warning messages.

Félix (Miadim) updated this revision to Diff 35350.Mar 18 2021, 6:57 PM
  • Cleaning: removed unused bmain an reports.
Harbormaster completed remote builds in B13596: Diff 35350.Mar 18 2021, 6:57 PM
Closed by commit rBbfad8deb0be0: VSE: Add new_meta RNA API function (authored by Felix Gremse (felix), committed by Richard Antalik (ISS)). · Explain WhyMar 20 2021, 1:42 AM
This revision was automatically updated to reflect the committed changes.
Richard Antalik (ISS) added a commit: rBbfad8deb0be0: VSE: Add new_meta RNA API function.

Revision Contents
Changeset List

PathSize
source/
blender/
makesrna/
intern/
67 lines
sequencer/
4 lines
intern/
27 lines

Diff 35342

View Options

source/blender/makesrna/intern/rna_sequencer_api.c

Loading...
View Options

source/blender/sequencer/SEQ_add.h

Loading...
View Options

source/blender/sequencer/intern/strip_add.c

Loading...