Early WIP on v2.55

[Michael Lobko-Lobanovsky]

Oh, then sure it's a deficiency in the Intel HD Graphics implementation of OpenGL driver.

We should make the glFlush()/glFinish() choice graphics vendor dependent...

[Patrice Terrier]

The glFinish function does not return until the effects of all previously called OpenGL functions are complete. Such effects include all changes to the OpenGL state, all changes to the connection state, and all changes to the framebuffer contents.

On my Intel it takes almost 90 seconds to complete all the effects, during all this time the application becomes unresponsive.

We should make the glFlush()/glFinish() choice graphics vendor dependent...

Of course, that is the best solution.

Added:
https://forums.intel.com/s/question/0D50P0000490CB8SAM/opengl-is-super-slow-on-windows-10-intel-hd-4000-and-earlier?language=en_US

https://www.opengl.org/discussion_boards/showthread.php/200367-An-Intel%28R%29-HD-Graphics-4000-War-Story

[Michael Lobko-Lobanovsky]

The glFinish function does not return until the effects of all previously called OpenGL functions are complete. Such effects include all changes to the OpenGL state, all changes to the connection state, and all changes to the framebuffer contents.

That's why I chose it for the task. I must be absolutely sure gP.hGL has completed its draw cycle before I proceed with a new iteration. Otherwise the viewport is likely to stay half-drawn.

If your Intel were running W7 rather than W10, you would see flicker with your glFlush() call pretty much like I'm seeing it without my glFinish(). It's because glFlush() wouldn't produce the desired effect.

So, it'll be either glFinish() if nVidia or ATi is detected or no such call at all in case of Intel (assuming it runs W10). We can postpone this issue till I (hopefully) have a discrete Intel video card later this year to test it.

[Michael Lobko-Lobanovsky]

So, please make it

                if (gP.graphicCard != INTEL)
                    glFinish(); // Complete redraw first then proceed (eliminate gP.hGL flicker)

in both cases.

[Patrice Terrier]

Please tell me if this code works on Seven, to keep the main caption enlighted when the help get visible and while switching the scrolling between the text and the mesh?


LRESULT CALLBACK HelpProc(IN HWND hWnd, IN UINT uMsg, IN WPARAM wParam, IN LPARAM lParam) {
    HDROP hfInfo = { 0 };
    WCHAR szFile[MAX_PATH] = { 0 };
    LRESULT nRet = 0;

    switch (uMsg) {
    case WM_NCACTIVATE:
        if (wParam) { skRestoreCaption(gP.hHelp, 0); }
        break;

    case WM_DROPFILES:
        hfInfo = HDROP(wParam);
        if (DragQueryFile(hfInfo, 0xFFFFFFFF, NULL, 0) > 0) {
            if (DragQueryFile(hfInfo, 0, szFile, MAX_PATH)) {
                ProccessCommandline(szFile);
            }
        }
        DragFinish(hfInfo);
        break;
    }
    return DefWindowProc(hWnd, uMsg, wParam, lParam);
}

[Michael Lobko-Lobanovsky]

Yes Patrice,

Your fix works flawlessly for me; the caption stays in focus, and the LED light, green, no matter what I do scrolling the help or panning/orbiting the model.

[Patrice Terrier]

Good, thank you.

[Patrice Terrier]

My friend

What are exactly the features that you want me to cleaned up from the version 2.75 release?

What are the new features do you think are missing from the documentation?
and to help my old brain, could you give me a short list of the new features that have been added since version 2.54.
(map_Ke, map_Ns, rotating lights, multiple shaders, model slicer, toast messages, transparent help, etc.)

Thank you!

[Michael Lobko-Lobanovsky]

Patrice,

1. I can't tell you exactly because I don't remember what code version you have. Please send me your sources tonight to WinDiff them, and I'll clean them up by tomorrow morning.

2. The new features are (not in the order of importance):

- transparent help and toast window help
- full OpenGL driver/GLSL version info
- model reload and unload
- load progress timer
- selectable fields of view
- emissive color mapping
- specular exponent mapping
- design modes:
     - texture inspection (Main->Inspect: 8 per-texture viewers + UV checker)
     - geometry inspection (Main->Inspect: material surface, color coded wireframe overlay, color coded meshes, vertex normals/tangents/bitangents)
     - mesh info tips
     - Z-axis rotation
     - bounding volumes
     - scene grid
     - model slicer
- 6 fancy lighting/shading modes (Main->Renderer)
- animated
     - lights
     - model reset
- not counting numerous bug fixes here and there...

Quite an impressive list for a minor version, isn't it?

[Michael Lobko-Lobanovsky]

Patrice,

The verified sources are attached below. A couple bugs fixed. (material initialization is definitely not your strongest point! )

The files have been checked for smooth compilation.

A few obsolete files are marked with a _WTF postfix.

We have differences in Help.h. I've included my version in the zip under the name of !!!MY_Help!!!.h. Please WinDiff them and advise which one I should use, yours or mine.

[Patrice Terrier]

Thank you very much, we are back in sync!

Here is the correct Help.h to use, yours didn't had the latest changes, especially to keep the caption highlighted while switching between GL for zooming and help for scrolling text with the middle mouse.

[Michael Lobko-Lobanovsky]

Thank you Patrice!

Strange as it may seem, my Help.h worked perfectly for me under W7 as far as scrolling and caption focus were concerned. I did make a few changes in it following your advice in this thread, after all.

But I'll be using the version that you attached above.

[Patrice Terrier]

Version 2.75 has been released.

I have reworked all the .png files from the resource folder with TruePNG, for the purpose of size reduction.

[Michael Lobko-Lobanovsky]

Thank you!

I've DL'ed the zip for the sake of getting the new PNG's.

You seem to have saved around 5MB of folder footprint.

[Michael Lobko-Lobanovsky]

We've achieved the goals set forth in this thread, so I'm closing it down.