Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transform node crashes app when scale == 0 #1635

Open
Kakapio opened this issue Feb 29, 2024 · 7 comments
Open

Transform node crashes app when scale == 0 #1635

Kakapio opened this issue Feb 29, 2024 · 7 comments
Labels
Crash A panic which crashed the editor Good First Issue Good for newcomers Rust Involves Rust programming for the backend

Comments

@Kakapio
Copy link
Contributor

Kakapio commented Feb 29, 2024

This is primarily an issue when dragging the scale parameters to adjust them in the UI. When the value for scale hits 0 on either x or y, Graphite crashes. I'm gonna look into this myself and try to patch it up (since I'm new 馃槃) but I'm making an issue to document it.

My solution is to just ignore an input of 0 in the backend. Let me know if that's too sloppy.

@Keavon Keavon added Good First Issue Good for newcomers Rust Involves Rust programming for the backend Crash A panic which crashed the editor labels Feb 29, 2024
@Kakapio
Copy link
Contributor Author

Kakapio commented Mar 1, 2024

issue only seems to occur when a TransformNode -> RepeatNode...

@devharsh2k4
Copy link

any updates?

@Kakapio
Copy link
Contributor Author

Kakapio commented Mar 5, 2024

any updates?

There are several upcoming refactors so this issue is being put aside for now.

@Keavon
Copy link
Member

Keavon commented Mar 5, 2024

Yeah, it's potentially blocked by @TrueDoctor's work on #1558

@0HyperCube
Copy link
Member

0HyperCube commented Mar 5, 2024

As far as I can tell (please include the actual panic message in future) this is probably from failing to inverse a transform matrix containing 0 in debug mode? We likely need to special case the zero transform such that we don't end up inverting.

@Keavon / @Kakapio I struggle to see how this is relevant to a change in the pivot code.

@Keavon
Copy link
Member

Keavon commented Mar 5, 2024

Thanks for chiming in @0HyperCube, perhaps this doesn't need to be blocked then (it just needs to have a zeroness check in the correct place, like where you suggested with the inverse transform matrix).

@Kakapio
Copy link
Contributor Author

Kakapio commented Mar 7, 2024

As far as I can tell (please include the actual panic message in future) this is probably from failing to inverse a transform matrix containing 0 in debug mode? We likely need to special case the zero transform such that we don't end up inverting.

@Keavon / @Kakapio I struggle to see how this is relevant to a change in the pivot code.

Here's the full output, I think. This should be the correct log. But yes, will look into this some more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Crash A panic which crashed the editor Good First Issue Good for newcomers Rust Involves Rust programming for the backend
Projects
Status: Short-Term
Development

No branches or pull requests

4 participants