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

Improve lighting #1680

Merged
merged 3 commits into from
May 7, 2014
Merged

Improve lighting #1680

merged 3 commits into from
May 7, 2014

Conversation

pjcozzi
Copy link
Contributor

@pjcozzi pjcozzi commented May 6, 2014

Fixes #592 and good enough for #989. Does not fix #1578.

No, we can't increase the scope of this pull request.

@kring
Copy link
Member

kring commented May 7, 2014

This seems like a solid incremental improvement to me. Tests pass. It's worth a mention in CHANGES.md though.

@pjcozzi
Copy link
Contributor Author

pjcozzi commented May 7, 2014

"Solid" is a bit too generous, but OK.

Ready.

kring added a commit that referenced this pull request May 7, 2014
@kring kring merged commit 3b536c3 into master May 7, 2014
@kring kring deleted the lighting branch May 7, 2014 21:01
Sign up for free to 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.

Abrupt lighting change Odd Columbus View Lighting...
2 participants