-
Notifications
You must be signed in to change notification settings - Fork 2
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better support for Tensorbackeds #29
Comments
I tend to prefer I find it more natural for me as a programmer to look for what I can do from a class using the autocompletion. For the second one, what about: |
@jcgarnier I agree for the first one, but the second would need to implement the operations in the a.class... It needs to be something like @kaifox proposed or |
For the first one, I prefer the For the second point, with the .as() method, we would get |
As I mentioned to Michi:
... But probably opinions change until somebody gets to the point of implementing something |
Additional things, which should go in here:
Currently it is only in tensorbacked internals... somethime it still is useful outside. |
compose(ATensorbacked.class).from(aTensor);
vs.:
aTensor.as(ATensoribacked.class);
(usually, the latter one should be possible ... as long it does not require anything additional (e.g. a field)).
Further:
calculate(ATensorbacked.class).by(a).times(b);
(or from(a)?)
The text was updated successfully, but these errors were encountered: