Skip to content

Update var_filter.py#20

Open
sunrisehang wants to merge 1 commit intoShichenXie:masterfrom
sunrisehang:master
Open

Update var_filter.py#20
sunrisehang wants to merge 1 commit intoShichenXie:masterfrom
sunrisehang:master

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.

2 participants