SynapseML
SynapseML copied to clipboard
microsoft
feat: R test gen
Related Issues/PRs
What changes are proposed in this pull request?
This PR introduces test generation for the R interface to SynapseML, similar to that for Python and Dotnet.
Briefly describe the changes included in this Pull Request. We generate tests for models in the R language that call the generated R functions that in turn call Scala code.
How is this patch tested?
We generate tests and run them.
Does this PR change any dependencies?
- [ ] No. You can skip this section.
- [x] Yes. Make sure the dependencies are resolved correctly, and list changes here. We have new dependences on R packages jsonlite and mlflow. Dependences on sparklyr and testthat were already present.
Does this PR add a new feature? If so, have you added samples on website?
- [x] No. You can skip this section.
- [ ] Yes. Make sure you have added samples following below steps.
AB#1898553
Hey @niehaus59 :wave:! Thank you so much for contributing to our repository :raised_hands:. Someone from SynapseML Team will be reviewing this pull request soon. We appreciate your patience and contributions :100:!
In reply to: 1196031249
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
![azure-pipelines[bot] avatar](https://avatars.githubusercontent.com/in/9426?v=4 "Published by a pub.dev verified publisher"){kind=small}
Codecov Report
Merging #1586 (169190b) into master (1381db5) will decrease coverage by
11.33%. The diff coverage is79.05%.
@@ Coverage Diff @@
## master #1586 +/- ##
===========================================
- Coverage 85.99% 74.65% -11.34%
===========================================
Files 287 272 -15
Lines 15104 14230 -874
Branches 756 739 -17
===========================================
- Hits 12988 10624 -2364
- Misses 2116 3606 +1490
| Impacted Files | Coverage Δ | |
|---|---|---|
| .../microsoft/azure/synapse/ml/cognitive/OpenAI.scala | 44.37% <ø> (-36.88%) |
:arrow_down: |
| ...oft/azure/synapse/ml/cognitive/TextAnalytics.scala | 32.14% <ø> (-51.59%) |
:arrow_down: |
| ...m/microsoft/azure/synapse/ml/codegen/CodeGen.scala | 100.00% <ø> (+1.33%) |
:arrow_up: |
| ...oft/azure/synapse/ml/codegen/DotnetWrappable.scala | 83.87% <ø> (-5.38%) |
:arrow_down: |
| .../azure/synapse/ml/core/utils/JarLoadingUtils.scala | 58.62% <ø> (ø) |
|
| ...crosoft/azure/synapse/ml/explainers/LIMEBase.scala | 92.30% <ø> (ø) |
|
| ...soft/azure/synapse/ml/explainers/TabularSHAP.scala | 89.47% <0.00%> (-2.42%) |
:arrow_down: |
| .../azure/synapse/ml/featurize/CleanMissingData.scala | 92.95% <0.00%> (-1.33%) |
:arrow_down: |
| ...crosoft/azure/synapse/ml/param/DataTypeParam.scala | 57.14% <ø> (ø) |
|
| ...ft/azure/synapse/ml/param/JsonEncodableParam.scala | 12.00% <0.00%> (-49.43%) |
:arrow_down: |
| ... and 80 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
