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

added support for readable model, invert hash and re-using a previous… #821

Merged
merged 10 commits into from
Mar 17, 2020

Conversation

eisber
Copy link
Contributor

@eisber eisber commented Mar 10, 2020

…ly trained VW Spark model

close #816

Please note I'm not sure how helpful invert hash is as the original input isn't passed along. The estimator only sees the Sparse/Dense Spark ML vector.

@eisber
Copy link
Contributor Author

eisber commented Mar 10, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov
Copy link

codecov bot commented Mar 10, 2020

Codecov Report

Merging #821 into master will decrease coverage by 1.71%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #821      +/-   ##
==========================================
- Coverage   85.32%   83.60%   -1.72%     
==========================================
  Files         186      186              
  Lines        8597     8560      -37     
  Branches      507      492      -15     
==========================================
- Hits         7335     7157     -178     
- Misses       1262     1403     +141     
Impacted Files Coverage Δ
...a/com/microsoft/ml/spark/vw/VowpalWabbitBase.scala 90.76% <100.00%> (+0.65%) ⬆️
.../microsoft/ml/spark/vw/VowpalWabbitBaseModel.scala 95.12% <100.00%> (+0.67%) ⬆️
...microsoft/ml/spark/automl/DefaultHyperparams.scala 0.00% <0.00%> (-100.00%) ⬇️
.../com/microsoft/ml/spark/automl/FindBestModel.scala 9.83% <0.00%> (-75.41%) ⬇️
...icrosoft/ml/spark/automl/TuneHyperparameters.scala 11.42% <0.00%> (-62.86%) ⬇️
...ala/com/microsoft/ml/spark/automl/ParamSpace.scala 0.00% <0.00%> (-60.00%) ⬇️
.../org/apache/spark/injections/RegressionUtils.scala 0.00% <0.00%> (-50.00%) ⬇️
...om/microsoft/ml/spark/train/AutoTrainedModel.scala 0.00% <0.00%> (-50.00%) ⬇️
.../microsoft/ml/spark/automl/HyperparamBuilder.scala 0.00% <0.00%> (-47.62%) ⬇️
...om/microsoft/ml/spark/automl/EvaluationUtils.scala 4.00% <0.00%> (-44.00%) ⬇️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7dd4670...d914231. Read the comment docs.

imatiach-msft
imatiach-msft previously approved these changes Mar 10, 2020
Copy link
Collaborator

@mhamilton723 mhamilton723 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a small req otherwise looks good

@eisber eisber self-assigned this Mar 16, 2020
@eisber
Copy link
Contributor Author

eisber commented Mar 16, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

mhamilton723
mhamilton723 previously approved these changes Mar 16, 2020
Copy link
Collaborator

@mhamilton723 mhamilton723 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you legend!

@eisber
Copy link
Contributor Author

eisber commented Mar 16, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit 9f15b6c into microsoft:master Mar 17, 2020
ocworld pushed a commit to AhnLab-OSS/mmlspark that referenced this pull request Mar 24, 2020
…revious… (microsoft#821)

* added support for readable model, invert hash and re-using a previously trained VW Spark model
*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vowpal wabbit in readable format
3 participants