Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request]: add an option or environment variable to use polar or polars-lts-cpu #4641

Open
1 task done
eipiem1 opened this issue Jan 25, 2025 · 1 comment
Open
1 task done

Comments

@eipiem1
Copy link

eipiem1 commented Jan 25, 2025

Is there an existing issue for the same feature request?

  • I have checked the existing issues.

Is your feature request related to a problem?

2025-01-24 11:14:22,922 INFO     13 ragflow_server log path: /ragflow/logs/ragflow_server.log, log levels: {'peewee': 'WARNING', 'pdfminer': 'WARNING', 'root': 'INFO'}
2025-01-24 11:14:40,554 INFO     14 task_executor_0 log path: /ragflow/logs/task_executor_0.log, log levels: {'peewee': 'WARNING', 'pdfminer': 'WARNING', 'root': 'INFO'}
2025-01-24 11:15:20,321 WARNING  13 /ragflow/.venv/lib/python3.10/site-packages/polars/_cpu_check.py:259: RuntimeWarning: Missing required CPU features.

The following required CPU features were not detected:
    avx, avx2, fma, bmi1, bmi2, lzcnt
Continuing to use this version of Polars on this processor will likely result in a crash.
Install the `polars-lts-cpu` package instead of `polars` to run Polars with better compatibility.

Hint: If you are on an Apple ARM machine (e.g. M1) this is likely due to running Python under Rosetta.
It is recommended to install a native version of Python that does not run under Rosetta x86-64 emulation.

If you believe this warning to be a false positive, you can set the `POLARS_SKIP_CPU_CHECK` environment variable to bypass this check.

Describe the feature you'd like

Add an option or environment variable to use polar or polars-lts-cpu for different cpu and OS

Currently it is configured in pyproject.toml and selected during build, with polar for x86-64 and polar-lts-cpu for Arm, configured incorrectly because the difference is not for x86-64/Arm but for difference instruction set supported. polar-lts-cpu is for broader range of cpus supported including some legacy cpus, while polar is for cpus with "modern" instruction set added, particularly polar is failed for some x86 cpu.

Describe implementation you've considered

No response

Documentation, adoption, use case

Additional information

No response

@KevinHuSh
Copy link
Collaborator

We've already changed polars-lts-cpu to polars-lts which has high compatibility.

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

No branches or pull requests

2 participants