http://www.miikahweb.com/feed.php MiikaHweb - Blender Commits By ISS http://www.miikahweb.com/ MiikaHweb - Blender Commits By ISS fi Thu, 17 Jun 21 19:30:12 +0000 MiikaHweb RSS contact at miikahweb.com <![CDATA[[78693d5] master: VSE: Refactor transform operator code]]> http://www.miikahweb.com/en/blender/git-logs/commit/78693d524c130b431c930eec90b8404ebad5e99a 78693d5
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Refactor transform operator code

Refactor function `freeSeqData` so it is readable.

One strip can have multiple transform operations defined. To prevent
processing strip multiple times, build `SeqCollection` and use
sequencer iterator instead of iterating `TransData` directly.

No functional changes.

Differential Revision: https://developer.blender.org/D11618

----------------------------------------
1 Path Changed]]>
Thu, 17 Jun 2021 02:04:01 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/78693d524c130b431c930eec90b8404ebad5e99aRichard Antalik
<![CDATA[[e7003bc] master: VSE: Reduce transform code complexity]]> http://www.miikahweb.com/en/blender/git-logs/commit/e7003bc9654b217e5938d64fb40094035735de9b e7003bc
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Reduce transform code complexity

Reduce complexity of sequencer transform code by removing recursivity.
This is possible by treating meta strips (mostly) as any other strip and
containing all transform code within SEQ_ functions.

Unfortunately internally meta strips still require special treatment,
but all complexity from code all over transform code seems to be
possible to contain within one function.

Functional change:
Previously adjusting handle of single image strip moved animation.
Now animation is not moved, which is behavior for all other strips.

Reviewed By: sergey, mano-wii

Differential Revision: https://developer.blender.org/D11493

----------------------------------------
7 Paths Changed]]>
Thu, 17 Jun 2021 01:42:04 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/e7003bc9654b217e5938d64fb40094035735de9bRichard Antalik
<![CDATA[[4891da8] master: Fix T88263: Incorrect image offset from old file]]> http://www.miikahweb.com/en/blender/git-logs/commit/4891da8ae22523c3ecb727f6f8be5647cbe0f51d 4891da8
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
Fix T88263: Incorrect image offset from old file

Versioning code for converting strip offset property doesn't work, when
property was animated and disabled or when crop was used.

When offset property is animated and offset is enabled, animation is
converted to be used with new transform design. When offset is disabled,
animation is left untouched. New transform design doesn't have option
to disable offset, and therefore old unconverted animation is used
instead of converted static value.

Remove animation from propery if it was unused.

Another issue was that both X and Y offset animation was being corrected
by factor caluclated for X channel.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11370

----------------------------------------
1 Path Changed]]>
Tue, 15 Jun 2021 22:44:37 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/4891da8ae22523c3ecb727f6f8be5647cbe0f51dRichard Antalik
<![CDATA[[1a5fa2b] master: VSE: Improve animation evaluation performance]]> http://www.miikahweb.com/en/blender/git-logs/commit/1a5fa2b319e06ebbd2666987aa11240a371dcc09 1a5fa2b
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Improve animation evaluation performance

Use lookup string callback function for `sequences_all` RNA property
`rna_SequenceEditor_sequences_all_lookup_string` using a GHash for faster lookups.

When names are changed or strips are added/removed the lookup is tagged invalid.
The next time the lookup is used it will rebuild it.

Reviewed By: sergey, jbakker

Differential Revision: https://developer.blender.org/D11544

----------------------------------------
14 Paths Changed]]>
Tue, 15 Jun 2021 22:29:17 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/1a5fa2b319e06ebbd2666987aa11240a371dcc09Richard Antalik
<![CDATA[[82c72cc] temp-attribute-processor: VSE: Remove seq->tmp usage from transform code]]> http://www.miikahweb.com/en/blender/git-logs/commit/82c72ccdbc0434e4d6592d0e6bb77b707e6176d6 82c72cc
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
VSE: Remove seq->tmp usage from transform code

This field was used for extend feature to get handle position of
metastrip children. Since D9972 extend feature works only on meta
strip itself, not it's children.
So `SEQ_transform_get_left_handle_frame()` second argument is always
false and can be removed.

Another instance of `seq->tmp usage` is hack to distinguish strips to be
shuffled, which is not covered by this patch.

Reviewed By: campbellbarton

Differential Revision: https://developer.blender.org/D10321

----------------------------------------
6 Paths Changed]]>
Mon, 07 Jun 2021 14:59:08 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/82c72ccdbc0434e4d6592d0e6bb77b707e6176d6Richard Antalik
<![CDATA[[01f2dad] temp-attribute-processor: VSE: Add refresh_all operator to all sequencer regions]]> http://www.miikahweb.com/en/blender/git-logs/commit/01f2dad596f46fa8e0ebe2ec8f3dc82752a15fac 01f2dad
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
VSE: Add refresh_all operator to all sequencer regions

This operator is needed in some cases to update image preview.
In workspaces with smaller timelines this is limiting, because users
need to first check that mouse cursor is in correct place, then press
CTRL+R shortcut.

----------------------------------------
1 Path Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/01f2dad596f46fa8e0ebe2ec8f3dc82752a15facRichard Antalik
<![CDATA[[ffa4c53] temp-attribute-processor: VSE: Remove JPEG reference from proxy panel]]> http://www.miikahweb.com/en/blender/git-logs/commit/ffa4c53d105114149efbdf4e2c3c03c51b3b7ec6 ffa4c53
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
VSE: Remove JPEG reference from proxy panel

Proxies doesn't use MJPEG codec anymore, but text still referenced it.

----------------------------------------
2 Paths Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/ffa4c53d105114149efbdf4e2c3c03c51b3b7ec6Richard Antalik
<![CDATA[[9e16740] temp-attribute-processor: Cleanup: Fix build warnings]]> http://www.miikahweb.com/en/blender/git-logs/commit/9e16740ca13eb7aac3cd6e65970f3690563e9e90 9e16740
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
Cleanup: Fix build warnings

----------------------------------------
3 Paths Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/9e16740ca13eb7aac3cd6e65970f3690563e9e90Richard Antalik
<![CDATA[[fc94932] temp-attribute-processor: VSE: Add strip-time intersection test function]]> http://www.miikahweb.com/en/blender/git-logs/commit/fc94932596d2321e0ed3c074be690e194b844f90 fc94932
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
VSE: Add strip-time intersection test function

Use SEQ_time_strip_intersects_frame function to test if strip intersects with frame.

Note: There are cases where this function should not be used. For example splitting
strips require at least 1 frame "inside" strip. Another example is drawing, where
playhead technically doesn't intersect strip, but it is rendered, because current
frame has "duration" or "thickness" of 1 frame.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11320

----------------------------------------
6 Paths Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/fc94932596d2321e0ed3c074be690e194b844f90Richard Antalik
<![CDATA[[97e363a] temp-attribute-processor: Cleanup: Strip duplication code]]> http://www.miikahweb.com/en/blender/git-logs/commit/97e363a06a5651193cae26a6d656aba052b71220 97e363a
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
Cleanup: Strip duplication code

Remove unused flag `SEQ_DUPE_ANIM` and code used by this flag.
Remove flag `SEQ_DUPE_CONTEXT` and refactor code, to split operator
logic from duplication code.
Reduce indentation level in for loop.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11318

----------------------------------------
4 Paths Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/97e363a06a5651193cae26a6d656aba052b71220Richard Antalik
<![CDATA[[7d47eea] temp-attribute-processor: Fix T57397: Movies are blurred after sws_scale]]> http://www.miikahweb.com/en/blender/git-logs/commit/7d47eea06d1e01a71fc0fd005d8954ecfd0ab6fe 7d47eea
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
Fix T57397: Movies are blurred after sws_scale

Images with 4:2:2 and 4:4:4 chroma subsampling were blurred when
`SWS_FAST_BILINEAR` interpolation is set for `anim->img_convert_ctx`.

Use `SWS_BILINEAR` interpolation for all movies, as performance is
not impacted by this change.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11457

----------------------------------------
1 Path Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/7d47eea06d1e01a71fc0fd005d8954ecfd0ab6feRichard Antalik
<![CDATA[[5dbe82f] temp-attribute-processor: FFmpeg: Update proxy settings]]> http://www.miikahweb.com/en/blender/git-logs/commit/5dbe82f12989e90555815d26a1aa1b58e23373ac 5dbe82f
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
FFmpeg: Update proxy settings

Changes in rBce649c73446e, affected established proxy codec preset.
Presets were not working and all presets were similar to `veryfast`.
Tunes are now working too, so `fastdecode` tune can be used. I have
measured little improvement, but I tested this only on 2 machines and
I have been informed that `fastdecode` tune does influence decoding
performance for some users.

Change preset from `slow` to `veryfast` and add tune `fastdecode`

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11454

----------------------------------------
1 Path Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/5dbe82f12989e90555815d26a1aa1b58e23373acRichard Antalik
<![CDATA[[9bd5dff] temp-attribute-processor: FFmpeg: Fix H264 lossless render not lossless]]> http://www.miikahweb.com/en/blender/git-logs/commit/9bd5dff7909815659cb71fb39755520d238fec42 9bd5dff
Branches : temp-attribute-processor
Author : Richard Antalik (ISS)
Committed By : Jacques Lucke (JacquesLucke)
----------------------------------------
FFmpeg: Fix H264 lossless render not lossless

While encoder parameters for lossless encoding are set correctly,
output is not lossless due to pixel format being set to
`AV_PIX_FMT_YUV420P` which is inherently lossy due to chroma subsampling.

This was reported in T61569 and was merged to T57397, but there were
2 bugs - one for encoding and one for decoding.

Set pixel format to `AV_PIX_FMT_YUV444P` when rendering lossless H264
files. This format isn't available in `codec->pix_fmts[0]` and it looks,
that it has to be hard-coded.

Reviewed By: sergey

Differential Revision: D11458

----------------------------------------
1 Path Changed]]>
Mon, 07 Jun 2021 14:58:32 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/9bd5dff7909815659cb71fb39755520d238fec42Richard Antalik
<![CDATA[[54ce344] master: VSE: Remove seq->tmp usage from transform code]]> http://www.miikahweb.com/en/blender/git-logs/commit/54ce344bc7cc9a0e1c34b328081cb90e41aca7b9 54ce344
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Remove seq->tmp usage from transform code

This field was used for extend feature to get handle position of
metastrip children. Since D9972 extend feature works only on meta
strip itself, not it's children.
So `SEQ_transform_get_left_handle_frame()` second argument is always
false and can be removed.

Another instance of `seq->tmp usage` is hack to distinguish strips to be
shuffled, which is not covered by this patch.

Reviewed By: campbellbarton

Differential Revision: https://developer.blender.org/D10321

----------------------------------------
6 Paths Changed]]>
Sun, 06 Jun 2021 01:05:45 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/54ce344bc7cc9a0e1c34b328081cb90e41aca7b9Richard Antalik
<![CDATA[[3e695a2] master: VSE: Add refresh_all operator to all sequencer regions]]> http://www.miikahweb.com/en/blender/git-logs/commit/3e695a27cdfad560d0b28e742cfa069d098200d6 3e695a2
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Add refresh_all operator to all sequencer regions

This operator is needed in some cases to update image preview.
In workspaces with smaller timelines this is limiting, because users
need to first check that mouse cursor is in correct place, then press
CTRL+R shortcut.

----------------------------------------
1 Path Changed]]>
Thu, 03 Jun 2021 17:58:36 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/3e695a27cdfad560d0b28e742cfa069d098200d6Richard Antalik
<![CDATA[[a1063fc] master: VSE: Remove JPEG reference from proxy panel]]> http://www.miikahweb.com/en/blender/git-logs/commit/a1063fc6c250a7a7016b11064403de5000c301dc a1063fc
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Remove JPEG reference from proxy panel

Proxies doesn't use MJPEG codec anymore, but text still referenced it.

----------------------------------------
2 Paths Changed]]>
Thu, 03 Jun 2021 17:44:00 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/a1063fc6c250a7a7016b11064403de5000c301dcRichard Antalik
<![CDATA[[5af7225] master: Cleanup: Fix build warnings]]> http://www.miikahweb.com/en/blender/git-logs/commit/5af72258161a6e903a5c526f2796db2a4e5bc483 5af7225
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
Cleanup: Fix build warnings

----------------------------------------
3 Paths Changed]]>
Wed, 02 Jun 2021 19:52:36 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/5af72258161a6e903a5c526f2796db2a4e5bc483Richard Antalik
<![CDATA[[925df8e] master: VSE: Add strip-time intersection test function]]> http://www.miikahweb.com/en/blender/git-logs/commit/925df8ef26a353497c8088657a82b6b8545c67c1 925df8e
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
VSE: Add strip-time intersection test function

Use SEQ_time_strip_intersects_frame function to test if strip intersects with frame.

Note: There are cases where this function should not be used. For example splitting
strips require at least 1 frame "inside" strip. Another example is drawing, where
playhead technically doesn't intersect strip, but it is rendered, because current
frame has "duration" or "thickness" of 1 frame.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11320

----------------------------------------
6 Paths Changed]]>
Wed, 02 Jun 2021 19:41:38 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/925df8ef26a353497c8088657a82b6b8545c67c1Richard Antalik
<![CDATA[[2ee575f] master: Cleanup: Strip duplication code]]> http://www.miikahweb.com/en/blender/git-logs/commit/2ee575fc1f2b13f083bc5996e20e7350570be546 2ee575f
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
Cleanup: Strip duplication code

Remove unused flag `SEQ_DUPE_ANIM` and code used by this flag.
Remove flag `SEQ_DUPE_CONTEXT` and refactor code, to split operator
logic from duplication code.
Reduce indentation level in for loop.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11318

----------------------------------------
4 Paths Changed]]>
Wed, 02 Jun 2021 19:41:17 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/2ee575fc1f2b13f083bc5996e20e7350570be546Richard Antalik
<![CDATA[[1f55786] master: Fix T57397: Movies are blurred after sws_scale]]> http://www.miikahweb.com/en/blender/git-logs/commit/1f557867916bd6ab43ced4b4fd651a7d1d11f8ac 1f55786
Branches : master
Author : Richard Antalik (ISS)
----------------------------------------
Fix T57397: Movies are blurred after sws_scale

Images with 4:2:2 and 4:4:4 chroma subsampling were blurred when
`SWS_FAST_BILINEAR` interpolation is set for `anim->img_convert_ctx`.

Use `SWS_BILINEAR` interpolation for all movies, as performance is
not impacted by this change.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D11457

----------------------------------------
1 Path Changed]]>
Wed, 02 Jun 2021 19:29:38 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/1f557867916bd6ab43ced4b4fd651a7d1d11f8acRichard Antalik