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
Taichi implicitly casts RangeFor boundaries to ti.i32, which is an unnecessary limitation and causes confusion when users explicitly specifies ti.i64 as RangeFor boundaries (#5217).
Ideally, backend's dtype limitation should not bother the frontend design. Unfortunately, we did forbiddened higher precision dtypes in certain cases due to concerns regarding the backends (RangeFor boundaries, ti.atan type conversion, etc..)
We should add a "dtype demotion" pass for each specific backend, and remove all these dtype conversions from the frontend.
The text was updated successfully, but these errors were encountered:
PR #5322 just added a warning message for this implicit conversion, which did not actually resolve this issue. Ideally we should convert the boundaries to int64 instead of int32.
Taichi implicitly casts RangeFor boundaries to ti.i32, which is an unnecessary limitation and causes confusion when users explicitly specifies ti.i64 as RangeFor boundaries (#5217).
Ideally, backend's dtype limitation should not bother the frontend design. Unfortunately, we did forbiddened higher precision dtypes in certain cases due to concerns regarding the backends (RangeFor boundaries, ti.atan type conversion, etc..)
We should add a "dtype demotion" pass for each specific backend, and remove all these dtype conversions from the frontend.
The text was updated successfully, but these errors were encountered: