You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Something is currently broken about the handling of mutable structs. One place I can isolate this is the following example, where Ref(Ref(x)) leads silently to wrong answers:
I presume that #1102 and/or #1103 caused this regression. There were, and are, astonishingly few tests of mutable structs. If anyone has other use cases they think ought to work, and can be compactly summarised to add to tests, that would be extremely helpful.
There are other mysterious bugs which seem like they are related to the handling of mutable structs, for which I don't have a compact example. It's likely that #1109 is related.
Bugs involving only immutable structs are not in scope here. There are some remaining leaks of ChainRules types, every time you fix one another appears... many are fixed by #1104
The text was updated successfully, but these errors were encountered:
I'm about to go this road, using mutable struct's. Docs said, not use if possible, but life is/will be much easier using them if they work as expected 😄 .
Something is currently broken about the handling of mutable structs. One place I can isolate this is the following example, where
Ref(Ref(x))
leads silently to wrong answers:That has been broken since at least 0.6.0. This version, with
Array(Ref(x))
, was not broken in 0.6.20, but currently gives a wrong answer:I presume that #1102 and/or #1103 caused this regression. There were, and are, astonishingly few tests of mutable structs. If anyone has other use cases they think ought to work, and can be compactly summarised to add to tests, that would be extremely helpful.
There are other mysterious bugs which seem like they are related to the handling of mutable structs, for which I don't have a compact example. It's likely that #1109 is related.
Bugs involving only immutable structs are not in scope here. There are some remaining leaks of ChainRules types, every time you fix one another appears... many are fixed by #1104
The text was updated successfully, but these errors were encountered: