Regression in 8348

Regression in 8348 - Messages

#1 Posted: 2/27/2023 11:47:23 AM
Martin Kraska

Martin Kraska

1222 likes in 2150 posts.

Group: Moderator

The attached sheet used to work and now doesn't.
The regression happened somewhere between build 7803 (works) and 7822 (doesn't work).
Also strange: the german error message even though the UI language was set to english.

Vektoriteration AEWP.sm (17 KiB) downloaded 64 time(s).
2023-02-27 15_31_23-SMath Solver - [Vektoriteration AEWP.sm].png
Martin Kraska Pre-configured portable distribution of SMath Studio: https://en.smath.info/wiki/SMath%20with%20Plugins.ashx
1 users liked this post
sergio 2/28/2023 6:12:00 AM
#2 Posted: 2/27/2023 1:47:27 PM
Jean Giraud

Jean Giraud

983 likes in 6866 posts.

Group: User

Confirmed: works fine SS 6179
#3 Posted: 2/27/2023 2:10:43 PM
overlord

overlord

554 likes in 1333 posts.

Group: Moderator

Is this the correct result?

2023-02-27_20-09.png
#4 Posted: 2/27/2023 3:41:02 PM
Martin Kraska

Martin Kraska

1222 likes in 2150 posts.

Group: Moderator

Yes, it is correct. Yet I consider the need to place eval() being a workaround for a bug. A similar workaround is to introduce an intermediate variable.

It seems that the error is related with broken logics of matrix operations. If I remove the factor in front of the matrix, then the function call works. Seems that at some point the evaluation logic thinks that the expression is not a matrix and complains in a not very helpful way.

Actually I found the culprit, it's the Ones() function. Davide, if you could ask that function to evaluate it's argument before refusing it, that would be fine.

2023-02-27 19_39_15-SMath Solver - [Vektoriteration AEWP.sm_].png
Martin Kraska Pre-configured portable distribution of SMath Studio: https://en.smath.info/wiki/SMath%20with%20Plugins.ashx
#5 Posted: 2/27/2023 4:03:35 PM
Jean Giraud

Jean Giraud

983 likes in 6866 posts.

Group: User

Wrote

Is this the correct result ?


... from your original document YES

#6 Posted: 2/27/2023 4:19:04 PM
overlord

overlord

554 likes in 1333 posts.

Group: Moderator

Wrote

1) Yet I consider the need to place eval() being a workaround for a bug.
2) Actually I found the culprit, it's the Ones() function.


1) I agree, putting line() or eval() sometimes is cumbersome. And unnecessary.
If necessity of them would be removed in all possibilities, it would be awesome.

2) You are 100% correct. I wonder how many of these little bugs out there.
This little change of code worked just fine.
2023-02-27_22-14.png

Regards
  • New Posts New Posts
  • No New Posts No New Posts