More PQSMods, LandControl and Rings#87
Open
TheWhiteGuardian wants to merge 3 commits intoKopernicus:masterfrom
Open
More PQSMods, LandControl and Rings#87TheWhiteGuardian wants to merge 3 commits intoKopernicus:masterfrom
TheWhiteGuardian wants to merge 3 commits intoKopernicus:masterfrom
Conversation
Also add the internal mod name because this is used by the Template node.
DeltaDizzy
requested changes
Aug 4, 2024
Member
DeltaDizzy
left a comment
DeltaDizzy
left a comment
There was a problem hiding this comment.
Overall Comments:
- Currently h3 headings do not generate links.
- Single-word headings do not need explicit tag declarations, though they are fine to leave in if you want
- The razor pages and routing additions are needed.
|
|
||
| AerialPerspectiveMaterial is a peculiar PQSMod. It appears to have no effect on the vertex height or color of the planet. Instead, it seems to partially configure the material of the planet's surface. It is suspected that the intent of this mod is to update certain shader parameters on every frame so as to correct any atmospheric effects. | ||
|
|
||
| ## Example {#Example} |
Member
There was a problem hiding this comment.
The anchor id ({#name}) is not needed, the C# side will autogenerate them based on the heading name.
|
|
||
| |Property|Format|Description| | ||
| |--------|------|-----------| | ||
| |atmosphereDepth|Decimal|The altitude of the atmosphere in m.| |
Member
There was a problem hiding this comment.
Use the full name (meters) in descriptions.
| |atmosphereDepth|Decimal|The altitude of the atmosphere in m.| | ||
| |globalDensity|Decimal|Unknown, presumed to be an atmosphere density multiplier.| | ||
| |heightFalloff|Decimal|The PQSMod appears to calculate atmospheric density at the altitude of the camera. This parameter seems to control the density falloff with altitude. The suspected behavior is `localDensity = exp(-heightFalloff * observerAltitude)`.| | ||
| |oceanDepth|Decimal|The depth of the ocean in m.| |
|
|
||
| The upshot is that parameters B and C essentially control the smoothness of the interpolation (possibly at the expense of actually passing through the height values recorded in the heightmap). Careless configuration may introduce unwanted artifacts, such as rippling effects around high contrast edges (IE what ought to be cliffs in the terrain) or blurring. | ||
|
|
||
| ## Example {#Example} |
| } | ||
| ``` | ||
|
|
||
| ## Properties {#Properties} |
| } | ||
| ``` | ||
|
|
||
| ## Parameters {#Parameters} |
| @@ -112,9 +161,49 @@ Each `LerpRange { }` node describes a range of numbers to encompass, or lerp ove | |||
| |endEnd|Decimal|The true end of the LandClass coverage. Coverage before this point is sparse, while coverage after this point is non-existent.| | |||
|
|
|||
| ## Scatters {#Scatters} | |||
Member
There was a problem hiding this comment.
Heading ID unnecessary (this slipped through the cracks when i purged them before)
| ## LerpRange {#LerpRange} | ||
| Each `LerpRange { }` node describes a range of numbers to encompass, or lerp over. These ranges are applied to each dimension. The image below describes the valid ranges for latitude and longitude, with a handy diagram at the bottom for a visual description of the coverage of the LandClass over a Float "dimension." Areas where coverage is not complete are determined by the `coverage___` properties. | ||
|
|
||
|  |
Member
There was a problem hiding this comment.
We should probably keep this image to illustrate it (I'll add it to the wiki repo directly)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.