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: explainers unit test runs into OOM #1093

Merged
merged 1 commit into from
Jun 19, 2021

Conversation

memoryz
Copy link
Contributor

@memoryz memoryz commented Jun 18, 2021

No description provided.

@memoryz
Copy link
Contributor Author

memoryz commented Jun 18, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 18, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov
Copy link

codecov bot commented Jun 18, 2021

Codecov Report

Merging #1093 (33158eb) into master (007a884) will increase coverage by 8.25%.
The diff coverage is 90.20%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1093      +/-   ##
==========================================
+ Coverage   77.26%   85.52%   +8.25%     
==========================================
  Files         230      232       +2     
  Lines        9317    10484    +1167     
  Branches      506      601      +95     
==========================================
+ Hits         7199     8966    +1767     
+ Misses       2118     1518     -600     
Impacted Files Coverage Δ
.../microsoft/ml/spark/automl/HyperparamBuilder.scala 47.61% <0.00%> (ø)
...soft/ml/spark/cognitive/TextAnalyticsSchemas.scala 100.00% <ø> (ø)
...a/com/microsoft/ml/spark/core/utils/RowUtils.scala 0.00% <0.00%> (ø)
...ala/com/microsoft/ml/spark/downloader/Schema.scala 61.53% <0.00%> (ø)
...icrosoft/ml/spark/io/binary/BinaryFileReader.scala 73.91% <ø> (ø)
...om/microsoft/ml/spark/io/http/SharedVariable.scala 66.66% <ø> (-16.67%) ⬇️
...com/microsoft/ml/spark/lightgbm/LightGBMBase.scala 92.85% <ø> (+8.54%) ⬆️
...crosoft/ml/spark/lightgbm/LightGBMClassifier.scala 91.01% <ø> (+2.12%) ⬆️
...icrosoft/ml/spark/lightgbm/LightGBMConstants.scala 100.00% <ø> (ø)
...microsoft/ml/spark/lightgbm/LightGBMDelegate.scala 50.00% <ø> (ø)
... and 379 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 725a92d...33158eb. Read the comment docs.

@memoryz
Copy link
Contributor Author

memoryz commented Jun 18, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 18, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 18, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz memoryz force-pushed the jasowang/unittest branch 3 times, most recently from dd2317d to 1e4f8cd Compare June 19, 2021 03:09
@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor Author

memoryz commented Jun 19, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit c84ab47 into microsoft:master Jun 19, 2021
@memoryz memoryz deleted the jasowang/unittest branch June 22, 2021 17:42
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.

None yet

2 participants