Skip to content

Commit

Permalink
Fix Hummingbird API changes (#418)
Browse files Browse the repository at this point in the history
* Hummingbird integration with lgbm
Add additional tests for Hummingbird

* Address flake8 errors

* we get hummingbird directly from github

* manually install torch

* roll back previous case

* skip HB tests for older versions of ORT

* fix reason and turn onnx into onnxruntime

* Addressing wenbingl comments

* fix to the git link

* add -f option to get hb

* add finx_links for pytorch in requirements
remove hummingbird_installed from utils
point to the onnxconveter-common branch with the hummingb_installed code

* Update convert.py

trigger pipeline

* point to the actual onnxconverter-common repository

* add torch to dev requirements
return the model from the hummingbird container
  • Loading branch information
interesaaat authored Sep 2, 2020
1 parent 8cf85d7 commit c60d0b5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion onnxmltools/convert/lightgbm/convert.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def convert(model, name=None, initial_types=None, doc_string='', target_opset=No
extra_config[constants.ONNX_INITIAL_TYPES] = initial_types
extra_config[constants.ONNX_OUTPUT_MODEL_NAME] = name
extra_config[constants.ONNX_TARGET_OPSET] = target_opset
onnx_model = convert(onnx_ml_model, "onnx", extra_config=extra_config)
onnx_model = convert(onnx_ml_model, "onnx", extra_config=extra_config).model

return onnx_model

Expand Down
1 change: 1 addition & 0 deletions requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,5 @@ wheel
xgboost
catboost
flake8
torch==1.5.1+cpu
hummingbird-ml

0 comments on commit c60d0b5

Please sign in to comment.