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

there is a error when run sml=Speedml(...) using the given example #8

Closed
leepand opened this Issue May 11, 2017 · 4 comments

Comments

2 participants
@leepand

leepand commented May 11, 2017

the error:

sml = Speedml(train='../input/train.csv', test='../input/test.csv', target = 'Survived',uid = 'PassengerId')
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/site-packages/speedml/init.py", line 46, in init
base.data_n()
TypeError: unbound method data_n() must be called with Base instance as first argument (got nothing instead)

thanks

@manavsehgal

This comment has been minimized.

Contributor

manavsehgal commented May 12, 2017

@leepand thanks for posting this. Seems like a Python 2.7 issue. I will investigate further and revert on this thread.

@leepand

This comment has been minimized.

leepand commented May 12, 2017

@manavsehgal thanks for your quick response, I have just solved the problem by adding @staticmethod on the data_n() function.

@manavsehgal

This comment has been minimized.

Contributor

manavsehgal commented May 12, 2017

Ok. that was a quick solution, thanks. I am exploring if that is the best solution for this problem and will fix it in the next release accordingly.

@manavsehgal manavsehgal added the bug label May 12, 2017

@manavsehgal

This comment has been minimized.

Contributor

manavsehgal commented May 12, 2017

@leepand Your solution worked well. Many thanks. The fix is now live. https://github.com/Speedml/speedml/releases/tag/v0.9.2

@manavsehgal manavsehgal added this to Release 0.9.2 in Speedml Core API May 12, 2017

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