-
Notifications
You must be signed in to change notification settings - Fork 420
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
ops::split() shorthand function is broken #357
Comments
I was able to work around this by building the operation without the helper function split() by doing the following:
where axis_zero is ops::constant(0,scope); however this doesnt close this issue since the helper function is broken and in the best case misleading. |
Edited : Default value is not irrelevant. |
FYI And the code was generated by tensorflow-op-codegen crate in this repository. |
Here is the
In this case, it seems we should not discard |
I cannot construct a split operation. when given the appropriate types in ops::split() the attribute num_split does not get assigned.
I'd reference the line but the file is too large for github, in my IDE it is 111704 in src/ops/ops_impl
there seems to be a discrepancy between num_split and split_dim as well.
It also doesnt seem like a great idea to have if statements for build_impl codegen where those attributes are mandatory without a default value for the node but I am still learning the codebase.
If anyone sees this please let me know how I can help otherwise I'll wing it.
The text was updated successfully, but these errors were encountered: