Use `isinstance(value, str)` instead of `value is str` for feature names in all places

sberbank-ai-lab / LightAutoML

LAMA - automatic model creation framework

Apache License 2.0

887 stars 92 forks source link

Closed resivalex closed 3 years ago

resivalex commented 3 years ago

Fixes #45

alexmryzhkov commented 3 years ago

@resivalex Ivan, sorry for my slow reply and thanks for your pull request!

Everything looks alright but I have one question - have you tested it on your case? Does it work as it should work?

Alex

resivalex commented 3 years ago

@alexmryzhkov Yes, I have. It works fine after these changes