Unnecessary base dependencies?

[what-in-the-nim]

I notice that there are additional base requirements added when bumping from 5.1.2 to 5.2.0

The PYYAML, PANDAS and NUMPY

pythainlp/setup.py

Lines 45 to 52 in 8819cfe

	requirements = [
	"backports.zoneinfo; python_version<'3.9'",
	"requests>=2.31",
	PYYAML,
	PANDAS,
	NUMPY,
	"tzdata; sys_platform == 'win32'",
	]

As I walk through the changes in v5.1.2...v5.2.0, I don't see any particular changes that we need to add it as a base requirement and it adds additional 79Mb when building Dockerfile. After investigating, it seems to be added in #1154 to fix the CLI problem. Since pythainlp is mostly use as a library (import) not as much as CLI. Should we add additional extra (pythainlp[cli]) instead to cope with this?

[github-actions]

Hello @what-in-the-nim, thank you for your interest in our work!

If this is a bug report, please provide screenshots and minimum viable code to reproduce your issue, otherwise we can not help you.

สวัสดี @what-in-the-nim ขอบคุณที่สนใจงานของเรา

ถ้านี่เป็นรายงานข้อผิดพลาด กรุณาแนบภาพหน้าจอ ข้อความแสดงข้อผิดพลาด และ โค้ดที่สั้นที่สุดเท่าที่จะทำให้เกิดปัญหา เพื่อที่เราจะสามารถช่วยเหลือได้

[what-in-the-nim]

[bact]

Agree that we can minimize the base dependencies.

We also currently have "requirements" in two places

• requirements list in setup.py
• requirements.txt

It's not ideal.

Looking forward, we should migrate to pyproject.toml and uses its [project.optional-dependencies] section.