-
Notifications
You must be signed in to change notification settings - Fork 3.8k
fixed tuple error #10216
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
fixed tuple error #10216
Conversation
mbrookhart
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but would you mind adding a test that hits your usecase?
c5cd449 to
03462a9
Compare
Added 3d input tests |
|
Looks like you have a lint problem. run |
It modified files I had nothing to do with. |
71befc1 to
6454e4d
Compare
|
@alnah005 it looks like your test failed in CI. do you mind taking a look? could you also clarify the PR title? |
I ran the tests on my end after your message and all the tests in the file are failing. I also ran the tests before my additions and they also failed. Here's the output I got after running without my additions: |
|
Appears to be a flaky CI thing. Please push an empty commit to go again |
areusch
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alnah005 i retriggered CI to see if this is another CI flake. can you also revert your changes to the python file permissions? looks like both files changed to 0755
Done! |
|
Thanks! Sorry for the long time to merge :/ |
tuple cannot be set because it is immutable. I was running a network that had a dense layer with 3D input. Very minimal fix!
@mbrookhart