segment-any-text / wtpsplit

Toolkit to segment text into sentences or other semantic units in a robust, efficient and adaptable way.
MIT License
625 stars 36 forks source link

[Bug] Pandas not listed in `install_requires`, resulting in import error #92

Closed mosheduminer closed 1 year ago

mosheduminer commented 1 year ago

When I tried to import wtpsplit to try it out, the program failed with an import error.

It appears that wtpsplit uses pandas (and imports it at the top level), but does not list it in setup.py, so it doesn't automatically get installed when wtpsplit is installed.

I can make a PR if you'd like (though I know it is a very small thing, lol).

bminixhofer commented 1 year ago

Good catch, and sorry for being so slow here. It's fixed in v1.2.0 (link).