Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Fix #225. Adapted shader alpha correction to the minimum number of sa… #236

Merged
merged 1 commit into from
Feb 8, 2016

Conversation

bilgili
Copy link
Contributor

@bilgili bilgili commented Jan 11, 2016

…mples

@@ -44,6 +44,8 @@ namespace livre

#define glewGetContext() GLContext::glewGetContext()

const uint32_t nDefaultSamplesPerRay = 512;
Copy link
Contributor

Choose a reason for hiding this comment

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

const static or move into anon namespace

@bilgili
Copy link
Contributor Author

bilgili commented Jan 12, 2016

please retest this

@bilgili
Copy link
Contributor Author

bilgili commented Jan 12, 2016

Retest this please.

@bilgili bilgili force-pushed the alpha_correction branch 3 times, most recently from 3a24fe6 to 1cbb78b Compare January 12, 2016 15:37
@bilgili
Copy link
Contributor Author

bilgili commented Jan 12, 2016

Do not merge it yet ! We have to check the alpha correction value is meaningful. @chevtche will check for it.

@bilgili
Copy link
Contributor Author

bilgili commented Feb 8, 2016

@eile - ping

@eile
Copy link
Contributor

eile commented Feb 8, 2016

+1

bilgili pushed a commit that referenced this pull request Feb 8, 2016
Fix #225. Adapted shader alpha correction to the minimum number of sa…
@bilgili bilgili merged commit c292091 into BlueBrain:master Feb 8, 2016
@bilgili bilgili deleted the alpha_correction branch February 15, 2016 12:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants