welly icon indicating copy to clipboard operation
welly copied to clipboard

Possible Bug in Project.data_as_matrix

Open hyperiongeo opened this issue 6 years ago • 3 comments

i have identified a bug in the Project.data_as_matrix. If you specify the parms and do not include any y_keys, then the y_train comes out as None, however, the line 442 in project.py has a blank return which generates an error.. eg. s,t = prj.data_as_matrix(['DT'],alias=alias)

gives an error 👍

TypeError Traceback (most recent call last) in () ----> 1 s,t = prj.data_as_matrix(['DT'],alias=welly_alias)

TypeError: 'NoneType' object is not iterable

if line 442 in project.py is changed to return X_train,__ the error disappears

hyperiongeo avatar Nov 18 '18 20:11 hyperiongeo

Feel free to contact me at hyperiongeo at gmail dot @com if you have any further questions.

Thanks

Corey

hyperiongeo avatar Nov 18 '18 20:11 hyperiongeo

Thanks Corey, I'll try to look at this soon.

(In the meantime, maybe try the new prj.df() to get a DataFrame. You'll need to be on the develop branch to get this.)

kwinkunks avatar Nov 19 '18 23:11 kwinkunks

Okay, thanks Matt.  I will pull the latest version down and take a look at the df(). Corey ________________________________________ Corey D. Hooge, M.Sc., P. Geoph. Senior Geophysicist

Email: mailto:[email protected] [email protected] Linkedin: http://ca.linkedin.com/in/chooge http://ca.linkedin.com/in/chooge Cell: tel:(403)%20828-4070 (403) 828-4070 From: mailto:[email protected] Matt Hall Sent: Monday, November 19, 2018 4:23 PM To: mailto:[email protected] agile-geoscience/welly Cc: mailto:[email protected] Hyperion ; mailto:[email protected] Author Subject: Re: [agile-geoscience/welly] Possible Bug in Project.data_as_matrix (#93)Thanks Corey, I'll try to look at this soon.

(In the meantime, maybe try the new prj.df() to get a DataFrame. You'll need to be on the develop branch to get this.)

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/agile-geoscience/welly/issues/93#issuecomment-440078801

hyperiongeo avatar Nov 19 '18 23:11 hyperiongeo