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

Fixes font glyph edge artifacts & scrolling issues #748

Merged
merged 6 commits into from
Jun 9, 2014

Conversation

JPierreS
Copy link
Contributor

@JPierreS JPierreS commented Jun 9, 2014

Implements a + X/Y 0.01 offset to d3d projection ortho. This may help fix rounding issues on many NVIDIA cards, in OPENGL. Tested extensively by myself on different systems, and by many ENIGMA users with AMD & NVIDIA cards, and some with INTEL and onboard. I tested it on 2 different types of NVIDIA cards on 2 different driver revisions from years old to most recent. Also tested with iGPU (Intel HD4000).

  • Fixes font glyph edge artifacts as seen by many NVIDIA users.
  • Fixes horizontal & vertical 1px gaps at the seams of scrolling backgrounds (tiled)
    as seen by many NVIDIA users, an issue that existed even without any offsets.

A discussion and testing can be found here:
http://enigma-dev.org/forums/index.php?topic=2022.0

So far positive feedback on phase 1 and phase 2, once it is established that there are no issues for AMD, NVIDIA and other users (iGPU, onboard) etc, I recommend this gets merged.

a .01 offset certainly is not big enough to cause issues. an EXE is provided in that topic to test font rendering, scrolling and other with the fixes applied.

Adds X/Y 0.01 offset to the projection ortho.
Corrects font glyph edge artifacts as well as a 1px gap at the 
seams of tiled scrolling backgrounds as was observed by many
users of NVIDIA cards.  This was extensively tested by many
users of NVIDIA, AMD, INTEL, iGPU & onboard cards. This does not
have any adverse effects on anything else or for people who never
experienced any issues.
Adds X/Y 0.01 offset to the projection ortho.
Corrects font glyph edge artifacts as well as a 1px gap at the 
seams of tiled scrolling backgrounds as was observed by many
users of NVIDIA cards.  This was extensively tested by many
users of NVIDIA, AMD, INTEL, iGPU & onboard cards. This does not
have any adverse effects on anything else or for people who never
experienced any issues.
@JPierreS JPierreS changed the title Fixes font glyph edge artifacts & scrolling issues Fixes font glyph edge artifacts & scrolling issues ** PLEASE DO NOT MERGE YET** Jun 9, 2014
@RobertBColton
Copy link
Contributor

Ok, dont comment this out though, remove it.

+//  After many tests, it was shown this part is not necessary.
+//  enigma::mv_matrix.translate(0.00f, 0.00f, 0.0f);

And you also need to remove it from GL3ModelStruct, the 3 line comment above it too.
https://github.com/enigma-dev/enigma-dev/blob/master/ENIGMAsystem/SHELL/Graphics_Systems/OpenGL3/GL3ModelStruct.h#L576

Once you do that, this can be merged.

Removed commented out code.
Removed lingering offset used in previous tests.
@JPierreS
Copy link
Contributor Author

JPierreS commented Jun 9, 2014

I removed the commented out code and the lingering offsets in the gl3modelstruct, guess since it was in the gl3modelstruct it did not bias any of my tests so far.
Done, all fixed now, I tested this one final time on my end, on all screen resolutions from 640x480 up to 1920x1080 native in window and FS, confirmed working.

@JPierreS JPierreS changed the title Fixes font glyph edge artifacts & scrolling issues ** PLEASE DO NOT MERGE YET** Fixes font glyph edge artifacts & scrolling issues Jun 9, 2014
@RobertBColton
Copy link
Contributor

This pull request has been tested by myself and other users on multiple hardware configurations, and everything appears to be working good now. This can be merged.

TheExDeus added a commit that referenced this pull request Jun 9, 2014
Fixes font glyph edge artifacts & scrolling issues ** PLEASE DO NOT MERGE YET**
@TheExDeus TheExDeus merged commit d0491ce into enigma-dev:master Jun 9, 2014
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.

3 participants