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

Higher default graphics quality on Android #13780

Merged
merged 1 commit into from Sep 7, 2023

Conversation

grorp
Copy link
Member

@grorp grorp commented Sep 4, 2023

This PR slightly (really) increases the default graphics quality on Android:

  • smooth_lighting enabled by default

  • enable_3d_clouds enabled by default

  • fps_max_unfocused increased to the same value as on desktop
    (On Android, there is a scrollbar in the pause menu, so a low fps_max_unfocused value is quite annoying.)

This doesn't have any impact on the framerate on my Android device (relatively new, but relatively cheap phone model). These changes are good for the first impression Android users get of Minetest. More changes in this direction could be made (see #12010), but this is a start.

To do

This PR is a Ready for Review.

How to test

Enjoy the higher-by-default graphics quality on Android.

@grorp grorp added @ Startup / Config / Util Android Feature ✨ PRs that add or enhance a feature labels Sep 4, 2023
settings->setDefault("sqlite_synchronous", "1");
settings->setDefault("map_compression_level_disk", "-1");
settings->setDefault("map_compression_level_net", "-1");
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

map_compression_level_disk and map_compression_level_net were redundant here since their default values are the same on desktop.

@grorp
Copy link
Member Author

grorp commented Sep 4, 2023

The CI failure is a dependency installation error on macOS.

Copy link
Member

@srifqi srifqi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested it and it works. I can barely tell any difference in performance (on Samsung Galaxy A72).

@grorp grorp merged commit 95056f9 into minetest:master Sep 7, 2023
12 of 13 checks passed
@grorp grorp deleted the android-graphics-defaults branch December 18, 2023 16:38
kawogi pushed a commit to kawogi/minetest that referenced this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants