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

fix: getTensorTypeFromSpark fails for Spark 3.3.0+ #607

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

memoryz
Copy link
Contributor

@memoryz memoryz commented Feb 23, 2023

The getTensorTypeFromSpark method fails for Spark 3.3.0+. This PR added "()" for pyspark.sql.types.DataType string representation.

Signed-off-by: Jason Wang <jasowang@microsoft.com>
@memoryz memoryz force-pushed the jasowang/sparkmltype branch from 10b7a1b to 0ea489e Compare February 23, 2023 08:06
@memoryz
Copy link
Contributor Author

memoryz commented Feb 23, 2023

Just found out this replicate #603.

@xadupre xadupre merged commit db71727 into onnx:main Feb 23, 2023
@xadupre xadupre mentioned this pull request Feb 23, 2023
@memoryz memoryz deleted the jasowang/sparkmltype branch February 23, 2023 17:34
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