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

Testing Fix torch v2 #484

Merged
merged 25 commits into from
Sep 6, 2023
Merged

Testing Fix torch v2 #484

merged 25 commits into from
Sep 6, 2023

Conversation

pgleeson
Copy link
Member

@pgleeson pgleeson commented Sep 1, 2023

No description provided.

davidt0x and others added 25 commits March 16, 2023 10:59
Looks like torch script compile was generating an
OnnxPad operation with None passed to constant_value
argument. This was causing a crash in the execution
engine. This was uncovered after fixing a bug of
handling prim::Constant Op correctly.
Little hack to pin ONNX ir_version to 8 if onnxruntime version
is less than 1.15. This should get fixed soon with update of
onnx package. See microsoft/onnxruntime#15874
Was producing an error as a 0 input (int) was being added to a float in
ONNX, which it didn't like
Minimal updates to work with modelspec v0.3 - onnx examples use floats
Seems like ONNX has switched to using / characters in
node names. This was causing issues with expression parsing
because output port ids are used in expressions.
Keras model serialized in examples/TensorFlow/Keras/kr_N_model.h5 is failing to load on new TF version. Getting:

TypeError: weight_decay is not a valid argument, kwargs should be empty  for `optimizer_experimental.Optimizer`.
Keras model serialized in examples/TensorFlow/Keras/kr_N_model.h5 is failing to load. Getting:

TypeError: weight_decay is not a valid argument, kwargs should be empty  for `optimizer_experimental.Optimizer`.

Can't reproduce locally, trying this suggested fix:

https://stackoverflow.com/questions/74667876/typeerror-weight-decay-is-not-a-valid-argument-kwargs-should-be-empty-for-opt
Re-enable full testing of package on 3.10
Looks like scikit-learn wheels aren't available for 3.11.
Getting compilation errors.
Update versions in github actions file
@pgleeson pgleeson merged commit ce187cb into feature/update_onnx Sep 6, 2023
32 of 33 checks passed
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