Skip to content
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

Update var_filter.py #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update var_filter.py #20

wants to merge 1 commit into from

Conversation

sunrisehang
Copy link
Contributor

@sunrisehang sunrisehang commented Apr 27, 2019

我直接在var_filter.py的文件里改了。
这样就可以避免,‘’如果有布尔列或者含有布尔列的数据时,生成列表时会出错的情况。‘’
我也已经测试过了。missing的情况不影响呀。

因为我这里,只是仅仅在数据处理的开始,将对象型和布尔型数据,转换成字符串式的对象型数据。对于整体的函数运行应该没影响。
并且,对于离散型数据的分箱,标准数据类型应该就是字符串吧?~
希望谢老师有空看看哈~~~如果想法不对,希望谢老师能够指正。

我直接在var_filter.py的文件里改了。
这样就可以避免,‘’如果有布尔列或者含有布尔列的数据时,生成列表时会出错的情况。‘’
我也已经测试过了。missing的情况不影响呀。
~~~因为我这里,只是仅仅在数据处理的开始,将对象型和布尔型数据,转换成字符串式的对象型数据。对于整体的函数运行应该没影响。
~~~并且,对于离散型数据的分箱,标准数据类型应该就是字符串吧?~
希望谢老师有空看看哈~~~如果想法不对,希望谢老师能够指正。
@ShichenXie
Copy link
Owner

如果别人不用var_filter呢

@sunrisehang
Copy link
Contributor Author

啊啊啊,怪不得谢老师,在每个函数的源码里,都有condition_fun.py文件中的那几个数据清洗的函数,领略到了~~~。
那就应该在condition_fun.py文件中修改,最好应该加一个函数,但是也没必要就是一条代码的事。但是谢老师觉得加到那个函数里好呢?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants