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
Saturn has no visible atmosphere #1395
Comments
The lua defs for gas giants are overridden here:
The colour is white and the density is 14 times Earth (1kg/m^3). I've checked the values sent to the shaders and they are ok..except for the fact that the molar mass of Earth air is used.. which is a lot heavier than hydrogen:) Saturn does have an atmosphere but it seems to be crushed by the high gravity to an extent with a scale height of ~3340m (low temps affect it badly as well). Density and colour are correct, the white towards the horizon should be the atmosphere. This will be fixed eventually when atmospheres have their own compositions. Jupiter should have these problems too. The solution, temporarily, may be to increase values sent to the shader for gas giants in Geosphere::Render() i.e. multiply the atmosScaleHeight by some factor X*(radius/Earth radius). X should be 3.0+ which should give scale height that's at least similar to Earth's scale height relative to its radius. |
The atmosphere equations for rendering are separate from those used in physics. The Venus issue is a typed error in an expression which didn't show up in Earth tests due to a bad coincidence with the units used and, is fixed in #1431. |
Just for further info, tried to gather hydrogen at a small gas giant, only 6 earth masses, thought it wouldn't suffer the same issue. Planet is Edwa C g - star coordinates -4,0,89 No atmosphere even down to 100m altitude. Zero pressure as well. |
Strangely enough, MJBN 834-6+5 d (-6,5,82) with a gravity of 23, radii 2.38 (Edwa C g had a gravity of 18, radii 1.8), could be refueled from, although had to be less than 1000m altitude for it to kick in. |
The problem concerning colour is known, and the 'proper' solution involves drawing the planet surface much deeper into the atmosphere solving being able to land on the surface as well.
This is more of an issue..Gas giants are hard-wired to have 14 kg/m^3 density at the surface. It could just be that there's no pressure due to surface temperature being close to zero (due to various issues) which will be dealt with. There should be a surface temperature read out in planet info. |
Gas giants are hard-wired to have 14 kg/m^3 Which is just bizarre and probably should be fixed at the same time as the rest. |
:D gas giants don't have a surface as such, so the value is just there to tweak things. The radius commonly quoted is the radius at which there is 1 atm of pressure. Luomu confirmed there was a 0 temperature issue which has been worked around in #1712. |
For a visual quickfix for this in
Then in
On Neptune it would look like this 16 km up; |
It's along the lines of what I wrote earlier, but:
Some gas giants may be so huge and have such high gravity that the scale heights may be small compared to their radii. This will make it hard to see the 'atmosphere'. The scaleheight issue has been mostly dealt with in the first part of the atmosphere branch, but as the complete atmosphere stuff will not be ready at least until the alpha after the next, it might be worthwhile to submit a workaround. This isn't the proper solution, and won't fix the pressure part of the problem, but I'll look into submitting a temporary fix today or tomorrow unless you get around to it first..
This has made the gas density much higher..it might end up obscuring the surface detail (selecting a planet and clicking ctrl+f10 will display it on the object viewer). Since we are actually drawing gas giant clouds with the surface colours, the plain atmosphere gets in the way, heh :) |
Sure go ahead, better than nothing until there's a proper solution I'd guess :)
Yeah I noticed, I only tested quickly on neptune and jupiter, but I can see how it obscures the surface on some seeds. |
Fixed by #2609. |
Notice altitude, mere 1500 m from the surface
The text was updated successfully, but these errors were encountered: