TEST-#6469: pin numexpr<2.8.5

[anmyachev]

What do these changes do?

Fix CI on master branch.

• first commit message and PR title follow format outlined here

  NOTE: If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

• passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
• passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
• signed commit with git commit -s
• Resolves numexpr 2.8.5 breaks CI #6469
• tests added and passing
• module layout described at docs/development/architecture.rst is up-to-date

[vnlitvinov]

Changes LGTM, but let's fix the English 🙃

[vnlitvinov]

LGTM, thanks @anmyachev!

[YarShev]

I think we should have the pin in setup.py too. If I installed modin from source, I would get an error like in the issue, right?

[anmyachev]

I think we should have the pin in setup.py too. If I installed modin from source, I would get an error like in the issue, right?

This is an optional dependency and should not be installed with Modin by default.

[YarShev]

pydantic is also an optional dependency but we have a pin for it in setup.py.

[anmyachev]

IIRC pydantic is installed with ray[default]. It is not optional for modin[ray].

[YarShev]

Anyway, if Modin doesn't work out of the box without the pin for numexpr, I think we should fix the things. Would you mind to open a PR so other maintaners take a look at the fix?

[anmyachev]

Anyway, if Modin doesn't work out of the box without the pin for numexpr, I think we should fix the things. Would you mind to open a PR so other maintaners take a look at the fix?

It's more quickly just ping @modin-project/modin-core here