The Wayback Machine - https://web.archive.org/web/20200927025409/https://github.com/Unity-Technologies/PostProcessing/pull/832
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

Fixed ReloadBaseTypes being called 2 times when entering play mode #832

Open
wants to merge 21 commits into
base: v2
from

Conversation

@PierreGac
Copy link

PierreGac commented Jul 8, 2019

It appears that when entering in playmode, the ReloadBaseTypes() is called 2 times:
instance.ReloadBaseTypes() => when the instance is null a new one will be created. Inside the ctor, at the end, ReloadBaseTypes() is called first. Then called again by the returned instance (instance.ReloadBaseTypes().
This fix will avoid such thing to happen.
The same issue occurs with EditorUtilities.ReloadDecoratorTypes but the previous fix cannot be applied (execution order is unclear)

bgolus and others added 2 commits Jun 4, 2019
Fixes an issue with MultiScaleVO showing incorrectly if XR and single pass stereo is enabled for a project, but the current camera not actually doing stereo rendering, like the scene view or when the HMD isn't active, or for "social" camera views.
…mode.

The same thing occurs with EditorUtilities.ReloadDecoratorTypes, but the fix cannot be applied in this context.
@PierreGac PierreGac force-pushed the PierreGac:v2 branch from ada56a3 to 7cba7a0 Sep 6, 2019
Pierre GAC added 19 commits Sep 6, 2019
…g/tree/patch-1' into v2

# Conflicts:
#	PostProcessing/Runtime/Effects/MultiScaleVO.cs
# Conflicts:
#	PostProcessing/Shaders/Builtins/ScreenSpaceReflections.hlsl

Added .gitattributes from ypapouin@cc0a269
Added pull request #854
…cd0d45

=> WARNING: When adding a custom effect, you need to add it to the PostProcessManager
Updated package.json
==> NOTE: Adding all changes by this commit will break the enable/disable behavior on the PostProcessVolume (need investigations)
Added changes from Tspk91@d3a2830
Revert package relative path of includes in shaders (to keep the good old .package approach working)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.