LGhassen / Deferred

Add deferred rendering to KSP
Other
78 stars 3 forks source link

Issues in RSS (and related mods) #29

Closed Clayell closed 4 months ago

Clayell commented 4 months ago

The terrain fails to load in the editor scene. image

Something strange happens with the shadow details when moving the camera around in the editor. image image image image

The terrain is very shiny in the flight scene. (seems fine in the KSC scene as far as I can tell) image image

Procedural Fairings turn white when the mouse is hovered near them, instead of the expected translucent. (this is probably the noted transparency issue) https://youtu.be/BGDT3lRFajA

KSP.log: https://drive.google.com/file/d/1bwIq-_UrPBlHx9xJ1oz3WuBvVBFn5qdO/view?usp=sharing

Installed mods:

Actual Sites: Airports (ActualSitesAirports 1.0.1) Advanced Jet Engine (AdvancedJetEngine v2.20.0) AlphaMensae's Modular Launch Pads (ModularLaunchPads 2.7.0) Approved Popular Pre-Launch Experiment (APPLE 1:1.1.9.1) ASET Consolidated Agency (ASETAgency v2.0.2) ASET Consolidated Avionics Pack (ASETAvionics v3.0.1) ASET Consolidated Props Pack (ASETProps v2.0.7) AtmosphereAutopilot (Fly-By-Wire) (AtmosphereAutopilot v1.6.1) AutoRCS (AutoRCS 1.1) B9 Aerospace Procedural Wings - Fork (B9-PWings-Fork 3:0.45.1) B9 Part Switch (B9PartSwitch v2.20.0) BahamutoD Animation Modules (BDAnimationModules 1:v0.6.7.1) BetterBurnTime (BetterBurnTime 1.10) BetterCrewAssignment (BetterCrewAssignment 1.4.1) Camera Tools continued (CameraTools v1.34.0) Chatterer (Chatterer 0.9.99) Chatterer Extended (ChattererExtended 0.6.2) ClickThrough Blocker (ClickThroughBlocker 1:2.1.10.21) Community Category Kit (CommunityCategoryKit v112.0.1) Community Resource Pack (CommunityResourcePack v112.0.1) Conformal Decals (ConformalDecals 0.2.14) Contract Configurator (ContractConfigurator v2.10.0.0) Cryo Tanks (CryoTanks 1.6.5) Cryo Tanks Core (CryoTanks-Core 1.6.5) Cryogenic Engines (CryoEngines 1:2.0.6) Custom Barn Kit (CustomBarnKit 1.1.22.0) Custom Parachute Message (CustomParachuteMessage v1.0.1) Custom Pre Launch Checks (CustomPreLaunchChecks 1.8.1.1) DE_IVAExtension (DE-IVAExtension v1.2.0) Debug Stuff (DebugStuff 1.7.0.0) Deployable Engines Plugin (DeployableEngines 1.3.1) Diazo's Landing Height Display (DiazosLandingHeight 2.3.0.1) Distant Object Enhancement /L (DistantObject v2.1.1.16) Distant Object Enhancement /L default config (DistantObject-default v2.1.1.16) Docking Port Alignment Indicator (DockingPortAlignmentIndicator 6.10.0.0) Dynamic Battery Storage (DynamicBatteryStorage 2:2.2.5.0) Editor Extensions Redux (EditorExtensionsRedux 3.4.5) Environmental Visual Enhancements Redux (EnvironmentalVisualEnhancements 3:1.11.7.2) Ferram Aerospace Research Continued (FerramAerospaceResearchContinued 3:0.16.1.2) Filter Extensions - Default Configuration (FilterExtensionsDefaultConfig 3.2.9) Filter Extensions - Plugin (FilterExtensions 3.2.9) Firespitter Core (FirespitterCore v7.17) Firespitter Resources config (FirespitterResourcesConfig v7.17) FreeIva (FreeIva 0.2.18.4) Hangar Extender (HangerExtenderExtended 3.6.0.2) Harmony 2 (Harmony2 2.2.1.0) Heat Control (HeatControl 0.6.1) HullcamVDS Continued (HullcamVDSContinued 0.2.2.1) Kerbal Alarm Clock (KerbalAlarmClock v3.14.0.0) Kerbal Atomics (KerbalAtomics 1:1.3.3) Kerbal Changelog (KerbalChangelog v1.4.2) Kerbal Engineer Redux (KerbalEngineerRedux 1.1.9.0) Kerbal Foundries2 (KerbalFoundriesContinued 2.4.8.18) Kerbal Joint Reinforcement Continued (KerbalJointReinforcementContinued v3.7.5.0) Kerbal Konstructs (KerbalKonstructs v1.8.7.0) Kerbal Renamer (KerbalRenamer v1.7.0.0) Kerbal Wind (KerbalWind 1:v1.0.2) Kerbalism (Kerbalism 3.19) Kerbalism - RealismOverhaul Config (Kerbalism-Config-RO v1.2.19.0) Kopernicus Planetary System Modifier (Kopernicus 2:release-1.12.1-206) KSA IVA Upgrade (KSAIVAUpgrade v1.6.7) KSC Floodlights Mk2 (KSCFloodlightsMk2 1.0) KSC Switcher (KSCSwitcher v2.1.0.0) KSP Community Fixes (KSPCommunityFixes 1.35.2) KSP Wheel (KSPWheel 1:0.16.14.33) KSPBurst (KSPBurst v1.5.5.1) Lunar Transfer Planner (LunarTransferPlanner v1.0.0) MagiCore (MagiCore 1.4.0.0) MechJeb 2 (MechJeb2 2.14.3.0) Minimum Ambient Lighting Updated (MinAmbLightUpd 1.2.6.2) ModularFlightIntegrator (ModularFlightIntegrator 1.2.10.0) Module Manager (ModuleManager 4.2.3) ModuleDepthMask (DepthMask 1.1.3) Navball Docking Alignment Indicator CE-2 (NavballDockAlignIndCE 1.1.1.3) NavHud (NavHudRenewed 1.4.0.5) Near Future Electrical (NearFutureElectrical 1.2.3) Near Future Electrical Core (NearFutureElectrical-Core 1.2.3) Near Future Exploration (NearFutureExploration 1.1.2) Near Future Launch Vehicles (NearFutureLaunchVehicles 2.2.0) Near Future Propulsion (NearFuturePropulsion 1.3.5) Near Future Solar (NearFutureSolar 1.3.2) Near Future Solar Core (NearFutureSolar-Core 1.3.2) Parallax (Parallax 2.0.8) Parallax - Stock Planet Textures (Parallax-StockTextures 2.0.8) Parallax - Stock Scatter Textures (Parallax-StockScatterTextures 2.0.8) Patch Manager (PatchManager 0.0.17.6) Pood's Milky Way Skybox (PoodsMilkyWaySkybox v.1.3.0) Precise Maneuver (PM) by Morse (PreciseManeuver 2:2.4.99.0-adoption) Procedural Fairings (ProceduralFairings 1:v6.6.0.0) Procedural Parts (ProceduralParts v2.5.9.0) QuickGoTo (QuickGoTo 1:1.4.0.10) QuickMute (QuickMute 1:1.4.0.8) RasterPropMonitor (RasterPropMonitor 1:v0.31.13.4) RasterPropMonitor Core (RasterPropMonitor-Core 1:v0.31.13.4) RCS Build Aid (RCSBuildAid v1.0.6) Real Antennas (RealAntennas v2.3.1) Real Fuels (RealFuels 1:v15.9.0.0) Real Plume (RealPlume 2:v13.3.2) Real Solar System (RealSolarSystem v20.1.2.0) Real Solar System Textures - 8192 x 4096 (RSSTextures8192 v18.6.1) RealChute for Stock (RealChuteForStock v1.4.9.4) RealChute Parachute Systems (RealChute v1.4.9.4) RealHeat (RealHeat v5.1) Realism Overhaul (RealismOverhaul v17.10.0.0) Realistic Progression One (RP-1) (RP-1 v3.9.0.0) Reentry Particle Effect Renewed (ReentryParticleEffect 1.9.1.1) ReStock (ReStock 1.4.5) ReStock Extra - Rigid Legs (RestockRigidLegs 1.4.5) ReStock+ (ReStockPlus 1.4.5) Retractable Lifting Surface Module (RetractableLiftingSurface 0.2.1.2) Reviva (Reviva 1.0.0) RO Capsules (ROCapsules v2.4.0.0) RO Engines (ROEngines v2.9.0.0) RO Heatshields (ROHeatshields v1.2.0.0) RO Library (ROLib v1.9.1.0) RO Solar (ROSolar v2.1.2.0) RO Tanks (ROTanks v2.10.0.0) ROEngines Extended (ROEnginesExtended 2.3.0) ROLoadingImages (ROLoadingImages v1.0.0.0) ROUtils (ROUtils v1.0.1.0) RP-1 TechTree Kompacted (RP1TechTreeKompacted 1.3) RSS CanaveralHD (RSS-CanaveralHD v1.0.1) RSS DateTime Formatter (RSSDateTimeFormatter v1.12.1.0) RSS Visual Enhancements - Low Resolution (RSSVE-LR 2.1.6) Scatterer (Scatterer 3:v0.0876) Scatterer Default Config (Scatterer-config 3:v0.0876) Scatterer Sunflare (Scatterer-sunflare 3:v0.0876) Shabby (Shabby 0.3.0.0) Shaddy (Shaddy v2.5) Show FPS (ShowFPS 1:0.3.1.3) SmokeScreen - Extended FX Plugin (SmokeScreen 2.8.14.0) Solver Engines plugin (SolverEngines v3.14.0) SpaceTux Library (SpaceTuxLibrary 0.0.8.6) Staged Animation (StagedAnimation v2.0) Superfluous Nodes (SuperfluousNodes 0.8) SXTContinued (SXTContinued 2:0.3.29.9) TAC Fuel Balancer (TacFuelBalancer v2.21.5.3) Test Lite (TestLite 0.3.5) TextureReplacer (TextureReplacer v4.5.3) Textures Unlimited (TexturesUnlimited 1.5.10.25) The Janitor's Closet (JanitorsCloset 0.3.8) Time Control (TimeControl 1:2.11) Toolbar Controller (ToolbarController 1:0.1.9.11) TooManyOrbits (TooManyOrbits 1.1.6.6) Transfer Window Planner - Fork (TransferWindowPlannerFork v1.9.1.0) Trim Indicator (TrimIndicator 1.12.0.0) TrimIndicators (TrimIndicators 0.1.0.4) TUFX (TUFX 1.0.7.1) TweakScale Redistributable (TweakScale-Redist v2.4.8.3) VABReorienter (VABReorienter 1.2) Ven's New Parts (VenStockRevamp-NewParts v1.16.0) Ven's Stock Part Revamp Core (VenStockRevamp-Core v1.16.0) Waterfall Core (Waterfall 0.9.0) WaypointManager (WaypointManager 2.8.4.2) Who Am I? (WhoAmI 1.4.0)

I would like to note that using deferred lighting instead of planetshine on RSS planets seems to work fine.

Clayell commented 4 months ago

Additional bug, the flight scene in the atmosphere shows strange black blobs of terrain, with seems to slightly correlate with the cloud position? Not sure. (using normal EVE clouds, not volumetric) The bug goes away when in the space flight scene.

https://youtu.be/7tTK-s5YfrY

Clayell commented 4 months ago

Additional bug, the flight scene in the atmosphere shows strange black blobs of terrain, with seems to slightly correlate with the cloud position? Not sure. (using normal EVE clouds, not volumetric) The bug goes away when in the space flight scene.

https://youtu.be/7tTK-s5YfrY

This bug seems to be active or inactive depending on the tufx profile I use, I'll have to test which specific setting removes the bug.

Clayell commented 4 months ago

This bug seems to be active or inactive depending on the tufx profile I use, I'll have to test which specific setting removes the bug.

Alright, so the black blobs directly correspond to the cloud position, and they also appear in the tracking station view. Additionally, it is definitely HDR that triggers this bug. When HDR is on, the bug appears, but when HDR is off, the bug vanishes. In the following pictures I have only changed the HDR setting.

image image

LGhassen commented 4 months ago

@Clayell for the black cloud artifacts, this can be fixed by setting the _ShadowFactor value in the cloud config "EarthCloudsLow" to 0.75 or any other value below 1 (default is 3), can you confirm this works? This seems to be independent from deferred btw and just related to hdr and cloud shadows.

I'm looking into the other issues.

Clayell commented 4 months ago

cloud config "EarthCloudsLow"

Where do I find this config?

LGhassen commented 4 months ago

In the EVE UI

LGhassen commented 4 months ago

I'll list procedurail fairings as incompatible and handle it separately. The other issues will be fixed in the next version apart from the shadows in the editor which is a stock issue I won't look into now.

Clayell commented 4 months ago

@Clayell for the black cloud artifacts, this can be fixed by setting the _ShadowFactor value in the cloud config "EarthCloudsLow" to 0.75 or any other value below 1 (default is 3), can you confirm this works? This seems to be independent from deferred btw and just related to hdr and cloud shadows.

Yep, can confirm that setting the value below 1 removes the black cloud artifacts, thanks! Perhaps some sort of check should be put in place with EVE and/or TUFX to turn that value down if HDR is enabled?

LGhassen commented 4 months ago

I will just cap the end result to be between 0-1

LGhassen commented 4 months ago

The other issues should be fixed now in yesterday's version, apart from the shadows in the editor, and the hdr cloud shadow issue which will be fixed separately in the next EVE/Scatterer update. Feel free to reopen this if you see any issues.

Clayell commented 4 months ago

The terrain fails to load in the editor scene. ... The terrain is very shiny in the flight scene. (seems fine in the KSC scene as far as I can tell)

Confirmed fixed.

Clayell commented 4 months ago

Edit: For the below pictures I had dithered transparency on, but it seems like dithered transparency doesn't affect the result at all?

Interestingly, the transparency stuff seems to depend on the TUFX profile. Using Default-Editor/Empty gives the normal white opaque stuff, but using Default-MainMenu/KSC/Tracking/Flight fixes the bug entirely. Looking into the TUFX settings, it seems to be another bug entirely controlled by HDR. (where turning HDR on fixes it) Might be worth looking into.

Default-Editor image

Default-Editor with HDR on image

(although it seems that the transparency in the fairing only shows radially attached parts? might be a stock bug idk)

Clayell commented 4 months ago

@LGhassen I think you force-closed this issue or something, I can't re-open it.

Additionally, al2me6 notes: (in regards to the TUFX HDR procedural fairings thing)

I'll take a look at exactly what PF is doing tonight. ... something is still messed up. If it's actually semitransparent then something is getting rendered in forward.

LGhassen commented 4 months ago

Not RSS related, I listed procedural fairings as incompatible in the compat list

Clayell commented 4 months ago

Not RSS related, I listed procedural fairings as incompatible in the compat list

Yes, but I think you should mention that turning HDR on removes the "white fairing" issue with procedural fairings.