Open mrocklin opened 6 years ago
Hey, we have an implicit contract that all functions must be called via modules, not as object methods. Try np.sum(np.exp(x)) + 1
. We haven't gotten around to enforcing this automatically or intelligently.
Thanks. That does seem to work.
Is this documented? Can I recommend that this also be included in an intelligent error message directing users to the right behavior?
On Mon, May 21, 2018 at 11:15 AM, Alex Wiltschko notifications@github.com wrote:
Hey, we have an implicit contract that all functions must be called via modules, not as objects methods. Try np.sum(np.exp(x)) + 1. We haven't gotten around to enforcing this automatically or intelligently.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/google/tangent/issues/73#issuecomment-390685147, or mute the thread https://github.com/notifications/unsubscribe-auth/AASszKeElZSYFViT4Wis3j1AtR39kvAZks5t0toSgaJpZM4UHIN9 .
You're correct that we need to make this specific thing better and clearer. I'll think on how best to do that. On Mon, May 21, 2018 at 11:18 AM Matthew Rocklin notifications@github.com wrote:
Thanks. That does seem to work.
Is this documented? Can I recommend that this also be included in an intelligent error message directing users to the right behavior?
On Mon, May 21, 2018 at 11:15 AM, Alex Wiltschko <notifications@github.com
wrote:
Hey, we have an implicit contract that all functions must be called via modules, not as objects methods. Try np.sum(np.exp(x)) + 1. We haven't gotten around to enforcing this automatically or intelligently.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/google/tangent/issues/73#issuecomment-390685147, or mute the thread < https://github.com/notifications/unsubscribe-auth/AASszKeElZSYFViT4Wis3j1AtR39kvAZks5t0toSgaJpZM4UHIN9
.
— You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/google/tangent/issues/73#issuecomment-390685981, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJ4j7xw74I5y3yOIDzSkj7_B9OroSOSks5t0trEgaJpZM4UHIN9 .
I'm probably doing something silly here, but I was confused by this error: