Regression: The start frame of NLA tracks can not be negative. #100918

New Issue

System Information
Operating system: Windows 10
Graphics card: NVIDIA GeForce RTX 2070

Blender Version
Broken: 3.3.0, branch: master, commit date: 2022-09-06 15:39
Worked: version: 3.2.0, branch: master, commit date: 2022-06-08 10:22

Caused by bd00324c26

Short description of error
Tracks start frame can not be negative. I the track is moved left, as soon as its left bound reach frame 0, it stays stuck here and the right bound is shrunk down.

Exact steps for others to reproduce the error

create a simple animation on an object
put this action as a track in the NLA editor
try to move this track to the left, past frame 0

Working version (3.2)

Bug (3.3)

Test File
nla_track.blend

This seems linked to this upgrade : https://developer.blender.org/rBbd00324c264c

**System Information** Operating system: Windows 10 Graphics card: NVIDIA GeForce RTX 2070 **Blender Version** Broken: 3.3.0, branch: master, commit date: 2022-09-06 15:39 Worked: version: 3.2.0, branch: master, commit date: 2022-06-08 10:22 Caused by bd00324c26 **Short description of error** Tracks start frame can not be negative. I the track is moved left, as soon as its left bound reach frame 0, it stays stuck here and the right bound is shrunk down. **Exact steps for others to reproduce the error** - create a simple animation on an object - put this action as a track in the NLA editor - try to move this track to the left, past frame 0 Working version (3.2) ![nla_track_ok_blender_3.2.gif](https://archive.blender.org/developer/F13469365/nla_track_ok_blender_3.2.gif) Bug (3.3) ![nla_track_bug_blender_3.3.gif](https://archive.blender.org/developer/F13469367/nla_track_bug_blender_3.3.gif) Test File [nla_track.blend](https://archive.blender.org/developer/F13469372/nla_track.blend) This seems linked to this upgrade : https://developer.blender.org/rBbd00324c264c

Added subscriber: @Alesk

#101368 was marked as duplicate of this issue

Added subscriber: @AdamJanz

Added subscribers: @thibaulltt, @BClark, @dr.sybren, @mano-wii

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

I'm not very familiar with the area.

If it really is a bug, I wonder how such trivial behavior went unnoticed in the review of D14658.
Cc @dr.sybren, @BClark, @thibaulltt

I'm not very familiar with the area. If it really is a bug, I wonder how such trivial behavior went unnoticed in the review of [D14658](https://archive.blender.org/developer/D14658). Cc @dr.sybren, @BClark, @thibaulltt

Oh I can see how we missed it because there were so many issues around this feature to get it working that we were testing the core clip behavior and not moving clips into negative ranges.

Does it change if "allow negative frames is on for the timeline?" I am downloading 3.3 now and will check but just wondering if that makes a difference.

Oh I can see how we missed it because there were so many issues around this feature to get it working that we were testing the core clip behavior and not moving clips into negative ranges. Does it change if "allow negative frames is on for the timeline?" I am downloading 3.3 now and will check but just wondering if that makes a difference.

Hi Brad, no that setting doesn't make any difference, I actually checked that a bit earlier today. In fact, "Allow Negative Frames" only permits the user to make the current frame negative. It does not permit a user to set the Start frame of the timeline to negative. EDIT: At least that is the behavior in 3.3 beta b3fc8206be released on August 9 (I have not tried the official 3.3 release yet).

Hi Brad, no that setting doesn't make any difference, I actually checked that a bit earlier today. In fact, "Allow Negative Frames" only permits the user to make the current frame negative. It does not permit a user to set the Start frame of the timeline to negative. EDIT: At least that is the behavior in 3.3 beta b3fc8206be42 released on August 9 (I have not tried the official 3.3 release yet).

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'

Added subscriber: @artutava

Happening here on 3.3 offical release. It's a very important bug... Wondering if opening files that were created in previous versions will affect the nla. @AdamJanz @BClark

Added subscriber: @lichtwerk

Hi there everyone, I've been on a bit of a crunch lately due to work. The fix is quite simple, I simply made a typo in the code for BKE_nlastrip_compute_frame_from_previous_strip().

In the NLA code, we have two sets of macros :

MINFRAME- [x] for the integer and floating-point macros for the smallest frame value which can be represented by the program (a.k.a the closest to 0),
MINAFRAME- [x] for the integer and floating-point macros for the lowest frame value which can be represented by the program.

In the function outlined above, I simply made a typo and forgot the A in the middle of the name :)

I'll make a patch, and see how fast I can make it available in a general release (could this maybe be a hotfix for the next patch version @lichtwerk ? maybe 3.3.1 ?)

Hi there everyone, I've been on a bit of a crunch lately due to work. The fix is quite simple, I simply made a typo in the code for `BKE_nlastrip_compute_frame_from_previous_strip()`. In the NLA code, we have two sets of macros : - `MINFRAME- [x]` for the integer and floating-point macros for the ***smallest*** frame value which can be represented by the program (a.k.a the closest to 0), - `MINAFRAME- [x]` for the integer and floating-point macros for the ***lowest*** frame value which can be represented by the program. In the function outlined above, I simply made a typo and forgot the `A` in the middle of the name :) I'll make a patch, and see how fast I can make it available in a general release (could this maybe be a hotfix for the next patch version @lichtwerk ? maybe 3.3.1 ?)

To everyone here, see : D15940

To everyone here, see : [D15940](https://archive.blender.org/developer/D15940)

In #100918#1415460, @thibaulltt wrote:
I'll make a patch, and see how fast I can make it available in a general release (could this maybe be a hotfix for the next patch version @lichtwerk ? maybe 3.3.1 ?)

Once committed, could be added here, yes #100749 (Blender LTS: Maintenance Task 3.3)

> In #100918#1415460, @thibaulltt wrote: > I'll make a patch, and see how fast I can make it available in a general release (could this maybe be a hotfix for the next patch version @lichtwerk ? maybe 3.3.1 ?) Once committed, could be added here, yes #100749 (Blender LTS: Maintenance Task 3.3)

This issue was referenced by 7c33d7b4b5

This issue was referenced by 7c33d7b4b5b334ed6d9c9585367982dc02671b52

Changed status from 'Confirmed' to: 'Resolved'

Wonderful! Thanks so much for the fix! Have a great week ahead guys. :-)

That was quick ! Thank you very much !

Very quick!! Is that a build for that already?

@artutava I don't think so. I would suggest checking back tomorrow at noon UTC for a daily master build over on https://builder.blender.org/download/daily/ to get the latest master build. However, be warned : those are technically beta builds and prone to crashing (not everything that ends up on master is stable, after all).

EDIT : while the link says daily builds of Blender are stable for the 3.3 version, I do not know which branch they originate from. If it's master, consider it beta. If it's blender-v3.3-release, it's way more stable since this is the LTS branch with a lot of fixes.

@artutava I don't think so. I would suggest checking back tomorrow at noon UTC for a daily master build over on https://builder.blender.org/download/daily/ to get the latest master build. However, be warned : those are technically beta builds and prone to crashing (not everything that ends up on master is stable, after all). EDIT : while the link says daily builds of Blender are stable for the 3.3 version, I do not know which branch they originate from. If it's `master`, consider it beta. If it's `blender-v3.3-release`, it's way more stable since this is the LTS branch with a lot of fixes.

Added subscriber: @CHARLES-MURRAY

Added subscribers: @item412, @deadpin

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Regression: The start frame of NLA tracks can not be negative. #100918