Skip to content

Switch to the package manager version of post-processing - #4

Merged
marian42 merged 3 commits into
marian42:masterfrom
IxxyXR:packagemanager-postprocessing
Nov 16, 2018
Merged

Switch to the package manager version of post-processing#4
marian42 merged 3 commits into
marian42:masterfrom
IxxyXR:packagemanager-postprocessing

Conversation

@andybak

Copy link
Copy Markdown
Contributor

No description provided.

@andybak

Copy link
Copy Markdown
ContributorAuthor

I'm getting a weird issue in play mode now that I can fix by switching camera and game view to "Display 2". Hopefully it's just a glitch on my install.

@andybak

Copy link
Copy Markdown
ContributorAuthor

Fixed. I'd accidentally added a second camera component.

@@ -0,0 +1,63 @@
%YAML 1.1

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you move the Posprocessing profile?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just went with the default location that Unity uses. It's a different profile because the old v1 profiles aren't compatible with v2. No harm in moving them back but I decided "go with the defaults" was the least controversial choice.

"com.unity.modules.unityanalytics": "1.0.0",
"com.unity.modules.unitywebrequest": "1.0.0",
"com.unity.modules.vr": "1.0.0",
"com.unity.modules.xr": "1.0.0"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you add these intentionally? Are they needed?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. Needed for Postprocessing v2 to compile without errors

@marian42
marian42 merged commit 1fbd0cf into marian42:masterNov 16, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@andybak@marian42