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

Negative values for TEXTUAL_FPS freeze the application. #5165

Closed
GEOEGII555 opened this issue Oct 23, 2024 · 2 comments · Fixed by #5192
Closed

Negative values for TEXTUAL_FPS freeze the application. #5165

GEOEGII555 opened this issue Oct 23, 2024 · 2 comments · Fixed by #5192

Comments

@GEOEGII555
Copy link

GEOEGII555 commented Oct 23, 2024

Setting TEXTUAL_FPS to -1 freezes the application, when it's supposed to ignore the value just like it does if you put 0 or a string into that environment variable.

How to reproduce:

[System.Environment]::SetEnvironmentVariable('TEXTUAL_FPS', '-1')
python3 -m textual

This issue was present in 0.52.1, and it's still present in the latest version (0.84.0). Someone thought about checking for non-numerical strings in the environment variable a long time ago, but no one thought about putting negative numbers in to this day?

изображение

Textual Diagnostics

Versions

Name Value
Textual 0.84.0
Rich 13.7.1

Python

Name Value
Version 3.12.7
Implementation CPython
Compiler MSC v.1941 64 bit (AMD64)
Executable C:\Users\GEOEGII555\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\python.exe

Operating System

Name Value
System Windows
Release 11
Version 10.0.22631

Terminal

Name Value
Terminal Application Windows Terminal
TERM Not set
COLORTERM Not set
FORCE_COLOR Not set
NO_COLOR Not set

Rich Console options

Name Value
size width=120, height=29
legacy_windows False
min_width 1
max_width 120
is_terminal True
encoding utf-8
max_height 29
justify None
overflow None
no_wrap False
highlight None
markup None
height None
Copy link

We found the following entry in the FAQ which you may find helpful:

Feel free to close this issue if you found an answer in the FAQ. Otherwise, please give us a little time to review.

This is an automated reply, generated by FAQtory

Copy link

Don't forget to star the repository!

Follow @textualizeio for Textual updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant