Closed m5signorini closed 1 year ago
Base: 85.35% // Head: 85.35% // Increases project coverage by +0.00%
:tada:
Coverage data is based on head (
ad415aa
) compared to base (3589f88
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
It will probably be a good idea to parametrize the output of the class, that is to add a Target type var to work with NDArrayInt and NDArrayStr. I include the change in this pull request.
I noticed every other classifier has the target restricted to NDArrayInt, should I update them in a similar way, using a TypeVar bound to Union[NDArrayInt,NDArrayStr]
, ? (In their respective pull requests)
Solution for nearest centroid and DTM for issue #480. Passes locally the test_classifier_classes. I think it is easier to modify the test script after updating all classifiers, while I locally check that the changes pass the test. Adding the classifier to the test script during each pull request may result in merge conflicts I guess.