Skip to content

Add rules for python in Development & Programming#408

Open
asyync1024 wants to merge 1 commit intoCachyOS:masterfrom
asyync1024:fix/python-priority
Open

Add rules for python in Development & Programming#408
asyync1024 wants to merge 1 commit intoCachyOS:masterfrom
asyync1024:fix/python-priority

Conversation

@asyync1024
Copy link
Copy Markdown
Contributor

Hi!
I decided to python to the rules because:

  • Before when python was launched, it would inherit -12 niceness, which makes it a direct competitor to Xorg etc.
  • Programs like lutris inherited the niceness too, which is bad as if you launch a game you obviously want it to have more resources than the launcher itself.

I have reduced the type to IN_DIFF as it's a neutral value, if you think it should be slightly more aggressive, then please correct me.
Thanks!

@Aarrayy
Copy link
Copy Markdown
Contributor

Aarrayy commented Mar 19, 2026

This one is tricky, not so long ago we added a rule for JAVA and it caused a ton of breakage.

@asyync1024
Copy link
Copy Markdown
Contributor Author

Yeah! This should only be merged after testing, of which I don't have any idea about. Though I think using IN_DIFF type is our safest bet here.

@Aarrayy
Copy link
Copy Markdown
Contributor

Aarrayy commented Mar 20, 2026

Or no rule at all.

@asyync1024
Copy link
Copy Markdown
Contributor Author

Or no rule at all.

If some type of testing conducted fails, then yes, that's our only option.

But I would still recommend to not make any quick decisions, python having -12 nicesness isn't correct.

Thanks!

@NIICKTCHUNS
Copy link
Copy Markdown
Contributor

NIICKTCHUNS commented Mar 21, 2026

This one is tricky, not so long ago we added a rule for JAVA and it caused a ton of breakage.

Java is indeed a hard one, things like Minecraft have their own process name, but others like WAKFU that I was going to add on another PR has the default name "java" so I couldn't add it, python on the other hand would influence some programs like Proton VPN GTK on Arch or even plasmoids like panel colorizer (at the moment Panel Colorizer has -6 nice, their process name is just python3 and the file is service.py )

@ptr1337
Copy link
Copy Markdown
Member

ptr1337 commented Mar 21, 2026

This is not good, if we really want to introduce compiler/python rules, then we need to disable ananicy-cpp out of the box again.

I know, for the general user expierence this could improve - but running Benchmarks on CachyOS will regress - massively.

@asyync1024
Copy link
Copy Markdown
Contributor Author

This is not good, if we really want to introduce compiler/python rules, then we need to disable ananicy-cpp out of the box again.

I know, for the general user expierence this could improve - but running Benchmarks on CachyOS will regress - massively.

Makes sense as one of my compiler and linker adding PRs was rejected because of that.

@asyync1024
Copy link
Copy Markdown
Contributor Author

I think a better way to handle these other rules like the python and compiler flags would be to add another repository named ananicy-rules-extra? And then create a submodule in the repo, one can recursively clone the repo if they want the compiler and other rules, otherwise just the normal rules.

This also allows making a separate package with the same repository name.

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

Successfully merging this pull request may close these issues.

4 participants