Skip to content

Sparse changes to allow for non-eager 'None' sizes - #422

Open
dmadisetti wants to merge 2 commits into
danielegrattarola:masterfrom
cemel-jhu:master
Open

Sparse changes to allow for non-eager 'None' sizes#422
dmadisetti wants to merge 2 commits into
danielegrattarola:masterfrom
cemel-jhu:master

Conversation

@dmadisetti

Copy link
Copy Markdown

Allows for GAT to be used with tensors of indeterminate size in single mode

@danielegrattarola

Copy link
Copy Markdown
Owner

Hey, thanks for the PR!
This is currently not passing the tests, I'll try to have a look into it to see if it can be fixed.

@dmadisetti

Copy link
Copy Markdown
Author

Weird. Not sure why this isn't failing locally. Looks like explicitly setting the signture type could work? I changed tf versions and will submit a fix

@dmadisetti

Copy link
Copy Markdown
Author

Works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants