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

Remove value-tuples from the public surface of the API #2501

Closed
rogancarr opened this Issue Feb 11, 2019 · 0 comments

Comments

2 participants
@rogancarr
Copy link
Contributor

rogancarr commented Feb 11, 2019

The public surface of the API contains value-tuples, which present a few problems:

  • We can't change the data we return in future releases (e.g. add functionality)
  • They don't play well with F#
  • Support in VS isn't great

These occur are in at least

  • CrossValidation (e.g. for each task)
  • TrainTestSplit: (e.g. for each task)

We should make sure that no value-tuples are returned and that they are not in the parameters of any public API.

Related to: #2487

@Ivanidzo4ka Ivanidzo4ka self-assigned this Feb 11, 2019

@Ivanidzo4ka Ivanidzo4ka added this to To do in Project 13 via automation Feb 12, 2019

@Ivanidzo4ka Ivanidzo4ka added this to In Progress in v0.11 Feb 12, 2019

@Ivanidzo4ka Ivanidzo4ka added the api label Feb 12, 2019

Project 13 automation moved this from To do to Done Feb 12, 2019

v0.11 automation moved this from In Progress to Done Feb 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment