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

Add scale/multiplier to TextureViewerLite node to fix rendering #2343

Merged

Conversation

DolphinDream
Copy link
Collaborator

Port the same preference settings for xy/scale into the TVL node to fix the location and the size of the rendering.

  • Code changes complete.
  • Ready for merge.

Before..

sv-tvl-renderfix-before

After...

sv-tvl-renderfix-after

Port the same preference settings for xy/scale into the TVL node to fix the location and the size of the rendering.
@DolphinDream
Copy link
Collaborator Author

Btw, is it just me or the two colors (the selected color in the Color input node and the output of the TVL node) are different ? Could it be the BGL rendering is not doing the same gl composition as the CI node ? The TVL output seems a bit darker.

@zeffii
Copy link
Collaborator

zeffii commented Dec 24, 2018

yep different, which to be honest I would expect in rgba mode, less in rgb. but i'll see how it behaves in 2.80

@zeffii
Copy link
Collaborator

zeffii commented Dec 24, 2018

in 2.80 it seems to swing the other way, it's waay brighter. Not sure what to do about that @DolphinDream

@DolphinDream
Copy link
Collaborator Author

in 2.80 it seems to swing the other way, it's waay brighter. Not sure what to do about that @DolphinDream

I wonder if it’s related to the color management module in Blender.

@DolphinDream
Copy link
Collaborator Author

any objections to merging this into master?

@zeffii
Copy link
Collaborator

zeffii commented Dec 24, 2018

nope. work away!

@DolphinDream DolphinDream merged commit ca22b47 into nortikin:master Dec 24, 2018
@DolphinDream DolphinDream deleted the textureViewerLiteRenderFix branch December 24, 2018 22:54
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.

2 participants