hoxo-m / densratio_py

A Python Package for Density Ratio Estimation
https://github.com/hoxo-m/densratio_py
Other
127 stars 30 forks source link

use len() instead of size method for list values #2

Closed msakai closed 5 years ago

msakai commented 5 years ago

According to its document uLSIF() takes list of float values as sigma_range and lambda_range, but it use their size method that are for numpy arrays but not for lists. This PR replaces size method with len() so that list arguments are processed as expected.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 95.673% when pulling 8855f9aa8840436b1a813a4dd20558ba7356e310 on msakai:len-instead-of-size into dd9106c5db45112b8047823588da29d38fa54bb0 on hoxo-m:master.

hoxo-m commented 5 years ago

I'm sorry. It has been fixed by another person. The cause is that my response was delayed. I really appreciate your contribution.