-
Notifications
You must be signed in to change notification settings - Fork 99
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
[WIP] Add converter for QuantileTransformer #705
base: main
Are you sure you want to change the base?
Conversation
This pull request introduces 1 alert when merging 1372d8b into 18d367d - view on LGTM.com new alerts:
|
Signed-off-by: xavier dupré <[email protected]>
Signed-off-by: xavier dupré <[email protected]>
This pull request introduces 1 alert when merging dc29fee into c503962 - view on LGTM.com new alerts:
|
Signed-off-by: xadupre <[email protected]>
Signed-off-by: Xavier Dupre <[email protected]>
model_onnx = convert_sklearn( | ||
model, "test", [("input", FloatTensorType([None, X.shape[1]]))] | ||
) | ||
self.assertTrue(model_onnx is not None) |
Check notice
Code scanning / CodeQL
Imprecise assert Note test
# opv = container.target_opset | ||
dtype = guess_numpy_type(operator.inputs[0].type) | ||
if dtype != np.float64: | ||
dtype = np.float32 |
Check notice
Code scanning / CodeQL
Unused local variable Note
No description provided.