salesforce / TransmogrifAI

TransmogrifAI (pronounced trăns-mŏgˈrə-fī) is an AutoML library for building modular, reusable, strongly typed machine learning workflows on Apache Spark with minimal hand-tuning
https://transmogrif.ai
BSD 3-Clause "New" or "Revised" License
2.24k stars 393 forks source link

Test to verify that custom metrics appear in model insight metrics #387

Closed tovbinm closed 5 years ago

tovbinm commented 5 years ago

Related issues No tests exist to verify that custom metrics appear in model insight metrics

Describe the proposed solution Added tests to verify that custom metrics appear in model insight metrics

Describe alternatives you've considered N/A

codecov[bot] commented 5 years ago

Codecov Report

Merging #387 into master will decrease coverage by 34.9%. The diff coverage is 71.42%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #387       +/-   ##
===========================================
- Coverage   86.85%   51.95%   -34.91%     
===========================================
  Files         336      336               
  Lines       10950    10955        +5     
  Branches      351      569      +218     
===========================================
- Hits         9511     5692     -3819     
- Misses       1439     5263     +3824
Impacted Files Coverage Δ
...m/salesforce/op/evaluators/EvaluationMetrics.scala 81.25% <100%> (-5.42%) :arrow_down:
...op/stages/impl/selector/ModelSelectorSummary.scala 85.71% <69.23%> (-6.84%) :arrow_down:
...sforce/op/stages/base/binary/BinaryEstimator.scala 0% <0%> (-100%) :arrow_down:
...la/com/salesforce/op/aggregators/Geolocation.scala 0% <0%> (-100%) :arrow_down:
.../scala/com/salesforce/op/test/FeatureAsserts.scala 0% <0%> (-100%) :arrow_down:
...la/com/salesforce/op/utils/io/avro/AvroInOut.scala 0% <0%> (-100%) :arrow_down:
.../salesforce/op/aggregators/FeatureAggregator.scala 0% <0%> (-100%) :arrow_down:
...n/scala/com/salesforce/op/readers/CSVReaders.scala 0% <0%> (-100%) :arrow_down:
...stages/base/sequence/BinarySequenceEstimator.scala 0% <0%> (-100%) :arrow_down:
.../scala/com/salesforce/op/readers/CSVDefaults.scala 0% <0%> (-100%) :arrow_down:
... and 170 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 07a4a73...bd6178a. Read the comment docs.

salesforce-cla[bot] commented 4 years ago

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Erica Chiu e***@s***.com. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request.