get_data rounds off REAL values #197

Activity

1. mfeurer commented on Feb 23, 2017

   

   mfeurer

   on Feb 23, 2017

   Collaborator

   More actions

   Try X, y = liver.get_data(target=liver.default_target_attribute, target_dtype=float). In the long run I suggest that you use tasks in order to retrieve the datasets you're looking at. Tasks usually handle this themselves.
2. joaquinvanschoren commented on Feb 25, 2017

   

   joaquinvanschoren

   on Feb 25, 2017

   ContributorAuthor

   More actions

   Thanks! Would be good to add that to the documentation, though?
3. mfeurer

   added

   bug

   on Sep 29, 2017
4. mfeurer commented on Oct 9, 2017

   

   mfeurer

   on Oct 9, 2017

   Collaborator

   More actions

   How about making this function private, and only exposing this correctly via the task object? People should rarely have to access raw data without a task.

   Thoughts @joaquinvanschoren @janvanrijn @amueller
5. janvanrijn commented on Oct 9, 2017

   

   janvanrijn

   on Oct 9, 2017 via email

   Member

   More actions

   Agreed. 2017-10-09 17:44 GMT+02:00 Matthias Feurer <[email protected]>:

   …

   How about making this function private, and only exposing this correctly via the task object? People should rarely have to access raw data without a task. Thoughts @joaquinvanschoren <https://github.com/joaquinvanschoren> @janvanrijn <https://github.com/janvanrijn> @amueller <https://github.com/amueller> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#197 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACL7-q6Plh88-G_AnyfgbNop8kUPeEnxks5sqj9egaJpZM4MJY6A> .
6. mfeurer

   added a commit that references this issue on Oct 12, 2017

   FIX #197, do not automatically cast target attribute

   This issue will close once commit 886a217 is merged into the 'main' branch.

   886a217
7. amueller commented on Oct 13, 2017

   

   amueller

   on Oct 13, 2017

   Contributor

   More actions

   closed by #352
8. amueller

   closed this as completedon Oct 13, 2017