Skip to content
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

ONNX export bug #1187

Open
brunoalvarez5 opened this issue Feb 20, 2025 · 0 comments
Open

ONNX export bug #1187

brunoalvarez5 opened this issue Feb 20, 2025 · 0 comments

Comments

@brunoalvarez5
Copy link

My comment is on the file parametric_umap.py

I've found an error when saving the model into an ONNX file, since it adds a ReLu at the end of the model. I've found the reason for this is that when defining the original encoder in the function forward it does not have this, however when it has to save it into ONNX first it passes the model to pytorch. So when it checks if the encoder is None and then defines it, in this definition it adds a ReLu and then exports the model like that.
I have attached screenshots of the two sections of the code I'm referring to for better clarity. (My solution was only commenting the line where the ReLu is defined) I really like this project, so could I perhaps open a fork to help with the project and fix this bug?
Thank you.

Image
Image

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

No branches or pull requests

1 participant