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

a tiny bit more forest #71735

Closed
wants to merge 1 commit into from

Conversation

NetSysFire
Copy link
Member

Summary

Content "Increase forest amount by a tiny bit"

Purpose of change

Massachusetts and New England in general are quite forested - but the ingame world is not, mostly due to gameplay reasons. However, I had a great™ idea: slowly increase the forest amount over weeks or months with multiple, small PRs adjusting the noise threshold. Until we notice issues or are simply satisfied with the amount of forest.
I proposed this on the devcord and have gotten a thumbs up by Maleclypse and otherwise no (negative) feedback.

Describe the solution

Decrease the forest noise threshold from 0.20 to 0.17. Leave the thick forest noise threshold as it is for now.

Describe alternatives you've considered

Testing

Loads and looks fine. The change is barely noticable, as expected.

Additional context

Could be a good and really simple backport candidate.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Feb 13, 2024
@NetSysFire NetSysFire added the Map / Mapgen Overmap, Mapgen, Map extras, Map display label Feb 13, 2024
@ZhilkinSerg
Copy link
Contributor

Instead of slowly and randomly changing stats you can generate a number of overmaps with different forest noise threshold values, count forest tile coverage for each value and compare with real world coverage (~60%).

Forest stats for 1337 overmaps of MA mod (which is based on real life data) give 56%:

CDDA_MA_ForestStats.xlsx

@ZhilkinSerg
Copy link
Contributor

Forest coverage for each overmap tile:

image

Sum of land_use_code_percentage Column Labels                                                                                                                                          
Row Labels 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 Grand Total
0                                                                                                   0,07 0,00                                     0,07
1                                                                                             0,00 0,30 0,36 0,43 0,49 0,22                                   1,80
2                                                                                         0,05 0,25 0,20 0,63 0,53 0,45 0,44 0,19                                   2,73
3                                                                                         0,38 0,48 0,30 0,51 0,69 0,74 0,59 0,50                                   4,19
4         0,10 0,70 0,53 0,71 0,53 0,64 0,56 0,49 0,42 0,39 0,37 0,31 0,28 0,23 0,18 0,15 0,06 0,05 0,00                                       0,01 0,24 0,26 0,37 0,50 0,57 0,73 0,68 0,83 0,48 0,00                                 11,41
5         0,39 0,72 0,46 0,73 0,75 0,91 0,89 0,89 0,90 0,85 0,88 0,86 0,83 0,83 0,82 0,86 0,54 0,97 0,90 0,91 0,87 0,81 0,77 0,66 0,50 0,63 0,59 0,50 0,46 0,46 0,46 0,38 0,34 0,28 0,27 0,22 0,14 0,11 0,30 0,48 0,16 0,25 0,58 0,71 0,74 0,72 0,68 0,53 0,08   0,03 0,01                           29,62
6         0,61 0,65 0,94 0,81 0,81 0,92 0,93 0,92 0,90 0,82 0,90 0,82 0,76 0,81 0,62 0,63 0,69 0,98 0,91 0,92 0,93 0,91 0,94 0,79 0,77 0,75 0,81 0,83 0,76 0,80 0,75 0,77 0,58 0,65 0,75 0,66 0,55 0,40 0,37 0,57 0,49 0,41 0,66 0,81 0,71 0,78 0,60 0,60 0,47 0,13 0,58 0,08                           37,01
7       0,03 0,86 0,91 0,99 0,53 0,83 0,98 0,96 0,90 0,82 0,87 0,76 0,76 0,71 0,53 0,60 0,65 0,81 0,93 0,94 0,84 0,75 0,93 0,88 0,76 0,92 0,92 0,80 0,82 0,83 0,85 0,78 0,84 0,70 0,70 0,82 0,60 0,41 0,19 0,27 0,55 0,49 0,61 0,84 0,86 0,63 0,82 0,59 0,67 0,75 0,68 0,43 0,17 0,00                         38,03
8       0,27 0,87 0,84 0,86 0,58 0,85 0,93 0,92 0,95 0,89 0,81 0,87 0,75 0,80 0,34 0,53 0,76 0,90 0,93 0,84 0,60 0,65 0,87 0,84 0,75 0,66 0,67 0,81 0,79 0,48 0,47 0,62 0,73 0,63 0,66 0,65 0,62 0,49 0,42 0,48 0,48 0,54 0,56 0,65 0,62 0,45 0,43 0,53 0,79 0,66 0,55 0,08                             34,73
9       0,50 0,88 0,72 0,73 0,91 0,92 0,95 0,91 0,92 0,78 0,82 0,82 0,84 0,81 0,55 0,63 0,91 0,95 0,92 0,87 0,75 0,87 0,92 0,87 0,78 0,76 0,68 0,69 0,67 0,69 0,33 0,54 0,67 0,49 0,61 0,52 0,59 0,65 0,71 0,43 0,35 0,50 0,53 0,68 0,41 0,33 0,20 0,18 0,18 0,02 0,00                               32,93
10     0,00 0,86 0,75 0,53 0,83 0,84 0,88 0,88 0,91 0,81 0,88 0,88 0,82 0,79 0,73 0,58 0,63 0,88 0,90 0,97 0,90 0,79 0,95 0,89 0,93 0,90 0,84 0,88 0,84 0,84 0,91 0,35 0,65 0,65 0,67 0,66 0,71 0,61 0,72 0,73 0,66 0,43 0,43 0,26 0,29 0,46 0,42 0,33 0,10 0,01                                   34,17
11     0,17 0,89 0,43 0,26 0,60 0,75 0,89 0,94 0,94 0,77 0,87 0,91 0,94 0,97 0,84 0,44 0,85 0,90 0,96 0,96 0,88 0,44 0,94 0,85 0,92 0,89 0,80 0,85 0,89 0,87 0,83 0,65 0,77 0,62 0,68 0,73 0,65 0,52 0,54 0,60 0,41 0,45 0,39 0,27 0,28 0,41 0,26 0,16 0,02 0,00                                   32,83
12     0,44 0,66 0,69 0,54 0,88 0,82 0,85 0,94 0,91 0,80 0,93 0,86 0,88 0,95 0,80 0,40 0,45 0,79 0,92 0,88 0,86 0,29 0,90 0,89 0,77 0,85 0,94 0,90 0,87 0,84 0,75 0,63 0,42 0,56 0,69 0,68 0,67 0,52 0,57 0,69 0,63 0,41 0,28 0,31 0,22 0,23 0,16 0,02                                       31,90
13     0,59 0,71 0,79 0,91 0,96 0,88 0,96 0,98 0,92 0,82 0,86 0,89 0,88 0,77 0,85 0,37 0,30 0,60 0,91 0,81 0,73 0,29 0,78 0,85 0,70 0,75 0,95 0,83 0,76 0,78 0,75 0,65 0,38 0,68 0,66 0,47 0,72 0,66 0,70 0,60 0,52 0,32 0,17 0,05 0,03 0,04 0,05                                         30,65
14   0,07 0,67 0,73 0,60 0,78 0,95 0,95 0,93 0,85 0,94 0,89 0,91 0,91 0,91 0,75 0,56 0,23 0,32 0,57 0,89 0,94 0,51 0,51 0,92 0,69 0,68 0,79 0,84 0,73 0,74 0,58 0,56 0,47 0,76 0,70 0,57 0,35 0,62 0,68 0,57 0,61 0,50 0,13 0,05 0,02 0,00 0,01 0,02 0,00 0,00                                     28,97
15   0,34 0,83 0,71 0,62 0,67 0,93 0,90 0,88 0,95 0,87 0,93 0,88 0,96 0,79 0,71 0,42 0,44 0,72 0,80 0,76 0,80 0,48 0,65 0,76 0,72 0,63 0,79 0,84 0,77 0,73 0,78 0,26 0,54 0,57 0,52 0,59 0,54 0,55 0,43 0,31 0,47 0,37 0,26 0,24 0,17 0,06 0,03 0,03 0,02 0,00                                     29,01
16   0,53 0,79 0,81 0,86 0,83 0,92 0,85 0,86 0,93 0,91 0,90 0,80 0,93 0,87 0,54 0,51 0,62 0,70 0,73 0,75 0,69 0,72 0,77 0,71 0,83 0,82 0,57 0,65 0,72 0,75 0,51 0,15 0,27 0,36 0,53 0,52 0,58 0,61 0,35 0,31 0,44 0,48 0,30 0,41 0,22 0,15 0,12 0,06 0,04 0,08 0,03                                   29,41
17 0,02 0,65 0,67 0,79 1,00 0,91 0,91 0,93 0,91 0,93 0,94 0,95 0,84 0,92 0,83 0,55 0,65 0,36 0,48 0,72 0,86 0,69 0,81 0,76 0,82 0,78 0,69 0,66 0,55 0,64 0,67 0,52 0,13 0,36 0,53 0,59 0,60 0,69 0,68 0,53 0,71 0,70 0,65 0,64 0,43 0,32 0,52 0,32 0,12 0,33 0,48 0,57 0,24                                 33,55
18 0,21 0,60 0,49 0,86 0,84 0,80 0,90 0,90 0,86 0,89 0,89 0,86 0,87 0,89 0,67 0,55 0,53 0,19 0,23 0,60 0,72 0,63 0,60 0,85 0,82 0,78 0,81 0,67 0,78 0,72 0,69 0,54 0,51 0,48 0,57 0,69 0,81 0,63 0,69 0,60 0,65 0,67 0,61 0,45 0,34 0,57 0,66 0,40 0,41 0,39 0,69 0,82 0,53 0,10                               34,52
19 0,53 0,76 0,65 0,88 0,81 0,93 0,92 0,93 0,80 0,86 0,96 0,83 0,88 0,71 0,55 0,60 0,51 0,21 0,38 0,28 0,51 0,78 0,70 0,75 0,85 0,82 0,80 0,82 0,77 0,61 0,72 0,64 0,66 0,49 0,55 0,57 0,60 0,54 0,60 0,60 0,62 0,65 0,69 0,41 0,51 0,40 0,58 0,37 0,58 0,46 0,49 0,67 0,66 0,36                               34,80
20 0,81 0,92 0,77 0,66 0,83 0,85 0,94 0,92 0,94 0,93 0,91 0,85 0,95 0,58 0,30 0,53 0,30 0,13 0,19 0,40 0,60 0,81 0,77 0,84 0,84 0,80 0,72 0,68 0,75 0,72 0,68 0,56 0,62 0,72 0,73 0,56 0,80 0,54 0,58 0,58 0,54 0,64 0,65 0,62 0,63 0,43 0,46 0,43 0,65 0,35 0,57 0,65 0,64 0,60 0,13                             35,66
21 0,85 0,87 0,72 0,54 0,74 0,90 0,93 0,94 0,95 0,91 0,95 0,89 0,85 0,66 0,48 0,48 0,39 0,25 0,25 0,42 0,64 0,83 0,78 0,85 0,88 0,81 0,87 0,70 0,61 0,67 0,68 0,63 0,93 0,72 0,63 0,52 0,63 0,72 0,64 0,62 0,49 0,55 0,68 0,55 0,69 0,76 0,56 0,16 0,46 0,54 0,65 0,68 0,74 0,67 0,49               0,09 0,33 0,09 0,03       37,56
22     0,01 0,04 0,10 0,17 0,26 0,30 0,34 0,41 0,45 0,45 0,42 0,58 0,29 0,32 0,29 0,36 0,24 0,35 0,48 0,65 0,67 0,70 0,71 0,67 0,77 0,85 0,78 0,62 0,49 0,33 0,80 0,83 0,76 0,77 0,66 0,75 0,61 0,71 0,63 0,69 0,67 0,54 0,61 0,67 0,52 0,55 0,53 0,66 0,72 0,55 0,74 0,68 0,26 0,01             0,01 0,21 0,02 0,46 0,16     28,89
23                           0,26 0,04                               0,02 0,02 0,16 0,36 0,33 0,30 0,19 0,19 0,10 0,15 0,29 0,56 0,47 0,59 0,57 0,73 0,78 0,65 0,54 0,58 0,68 0,54 0,68 0,42 0,07 0,05                   0,19 0,71 0,01   11,27
24                                                                                 0,16 0,45 0,50 0,66 0,65 0,66 0,76 0,65 0,58 0,68 0,70 0,78 0,70 0,54 0,13 0,02 0,01                 0,18 0,87 0,29   9,97
25                                                                                 0,10 0,51 0,38 0,70 0,76 0,58 0,50 0,67 0,68 0,74 0,76 0,69 0,72 0,46 0,61 0,70 0,40                 0,09 0,25 0,58   10,88
26                                                                                 0,02 0,25 0,70 0,76 0,74 0,66 0,38 0,56 0,70 0,55 0,71 0,79 0,77 0,82 0,86 0,67 0,70 0,01               0,00 0,02 0,65 0,08 11,41
27                                                                                   0,21 0,71 0,73 0,77 0,68 0,67 0,71 0,67 0,41 0,70 0,95 0,76 0,90 0,87 0,76 0,78 0,06                   0,35 0,13 11,83
28                                                                                   0,20 0,47 0,70 0,79 0,64 0,61 0,75 0,72 0,47 0,70 0,87 0,79 0,93 0,88 0,63 0,50 0,04                 0,03 0,44 0,14 11,29
29                                                                                   0,05 0,36 0,63 0,68 0,31 0,73 0,95 0,78 0,66 0,69 0,82 0,75 0,64 0,72 0,62 0,66 0,49 0,16         0,02 0,19 0,32 0,51 0,63 0,13 12,46
30                                                                                     0,00 0,27 0,33 0,25 0,84 0,93 0,83 0,68 0,70 0,76 0,76 0,35 0,37 0,26 0,76 0,69 0,70 0,60 0,48 0,13 0,24 0,52 0,60 0,59 0,60 0,36 0,03 13,64
31                                                                                       0,00 0,03 0,12 0,73 0,72 0,80 0,40 0,76 0,86 0,62 0,19 0,05 0,38 0,78 0,60 0,55 0,69 0,63 0,54 0,63 0,44 0,45 0,65 0,51 0,43 0,10 12,67
32                                                                                         0,03 0,20 0,69 0,70 0,56 0,12 0,56 0,53 0,34 0,00 0,04 0,48 0,62 0,49 0,61 0,55 0,45 0,22 0,30 0,18 0,12 0,03 0,01 0,04 0,03 7,89
33                                                                                           0,32 0,69 0,74 0,55 0,09 0,18 0,17     0,03 0,59 0,59 0,64 0,54 0,22 0,09 0,02 0,08         0,00   5,54
34                                                                                           0,31 0,61 0,77 0,64 0,10 0,02 0,06     0,05 0,61 0,40 0,49 0,24                 0,02   4,32
35                                                                                           0,20 0,54 0,58 0,57 0,13       0,01 0,18 0,10 0,03 0,03 0,00                 0,01   2,37
36                                                                                           0,06 0,07 0,22 0,05 0,00     0,11 0,40 0,06 0,01                           0,99
37                                                                                                   0,00 0,01 0,20 0,41 0,00 0,18 0,38 0,13                         1,32
38                                                                                                   0,08 0,26 0,00   0,17 0,76 0,76 0,48 0,04 0,03                     2,58
39                                                                                                         0,04 0,70 0,69 0,78 0,79 0,39 0,38               0,00 0,01   3,77
40                                                                                                       0,33 0,38 0,41 0,17 0,19 0,11 0,06 0,04     0,01         0,00 0,12   1,83
41                                                                                                       0,02 0,11                   0,08 0,06 0,21 0,10 0,33 0,72 0,06 1,69
42                                                                                                       0,13                       0,07 0,38 0,36 0,57 0,68 0,07 2,27
Grand Total 2,42 4,75 6,80 9,93 12,76 13,61 15,44 15,17 15,73 16,73 16,69 15,81 15,66 15,51 13,86 12,87 11,56 7,88 9,12 12,00 13,86 15,23 13,87 12,51 14,96 14,81 14,40 13,67 13,70 13,34 13,22 12,02 11,27 10,42 11,17 11,66 11,40 10,96 11,42 10,25 10,15 12,10 12,99 13,23 13,42 13,66 17,36 18,56 17,58 14,29 16,18 16,65 13,46 10,71 8,70 8,23 7,76 4,04 3,24 2,06 1,64 0,92 1,44 1,84 2,06 3,01 4,58 5,35 0,76 754,43

@NetSysFire
Copy link
Member Author

There is one problem with that though imo: Your proposed value is the mean forest value across all overmaps combined. A significant amount of the Massachusetts mainland is quite densely forested, excluding the coast (yet to be its own full biome ingame) and the big metropolitan area (implemented as part of #69912, so theres more city mapgen than forest noise there).
#69912 also messed with the noise, so the further west or north you get, the denser the forest, though admittedly I did not consider this for this PR.

So if we want a truly realistic value, we may actually need to exclude some MA overmap tiles from factoring into your calculation, most notably the big metropolitan area, coasts and maybe where the forest actually starts to get more dense.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 13, 2024
@ZhilkinSerg
Copy link
Contributor

You can code a gradient for forest noise threshold value - more forest to the west (90%), less forest to the east (0-1%) with an average of 60%, so it would resemble MA forest density.

@NetSysFire NetSysFire closed this Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants