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
Writing our own cross-backend API is something that is really annoying for us to do and also very annoying for a user to learn. It probably also has a ton of consequences in terms of whether GPU pipeline can work (remember the need for out= arguments), whether higher-level functions can be jitted, and how IDEs interact with the code. We should instead vendor something like https://data-apis.org/array-api-compat/ which conforms to a known common interface
These setattr calls that mutate the cb interface at file load time are a huge problem imo because the interface ends up with undocumented functions. We should accept the untidyness of either moving these rather specific functions into the cb, or wrap them in an if/else inside the user class itself
We currently do this only for the control interface, but should really consider applying the cb across the board and just setting the Isaac Sim frontend to use numpy/torch based on user choice.
This has huge implications on whether full GPU pipelining can be done at all, so we should be careful. Even just the current control-only conversion might mean that there is suddenly no acceptable way to put control on GPU in a reasonable way. It might effectively mean the whole torch conversion was entirely pointless.
The text was updated successfully, but these errors were encountered:
This is meant as a follow up to #1051
The text was updated successfully, but these errors were encountered: