Code review comment for lp:~kiithsacmp/stellarium/glexperiment

I'll look at those compilation bugs (don't have a Mac, though).

As for performance, there is visible decrease with the GL2 backend
(60-70% performance), but nothing that massive. On my new PC (i5 3570K
3.4GHz, Radeon HD6770, open source AMD drivers (Catalyst doesn't work
for Ubuntu 12.10 yet, but Catalyst with 12.04 had similar slowdown, at
higher FPS))

Also, try the GL1 backend (simplest way is to use --safe-mode) - on my machine
it's speed roughly matches trunk (most of time slightly - ~10% slower,
sometimes, e.g. planet drawing, faster)

Most likely there's a driver specific issue (what's the driver, GPU?
Is that on the Mac?) Maybe a profiler would help.

A profile might help but it usually only helps find CPU-bound problems,
it's unlikely this is the case here.

Also, any shader warnings? (show up during run time)

Going to compare FPS trunk<->refactored right now, will reply later
with results.

On 8/30/12, treaves <email address hidden> wrote:
> Review: Needs Fixing
>
> This simply fails to compile.
>
> Looking into it more, there are several issues:
> - GL_RGBA32F on Mac is GL_RGBA32F_ARB. It looks like this may be due to
> Apple not having updated their include files to reflect that it is now
> standard, and not experimental.
>
> - SolarSystem.cpp:937:12: error: variable length array of non-POD element
> type 'Vec4f' (aka 'Vector4<float>')
> - SolarSystem.cpp:940:22: error: variable length array of non-POD element
> type 'Mat4d' (aka 'Matrix4<double>')
>
> For those last two, you need to do heap allocation, or refactor the code to
> use a Qt structure (QVector perhaps?).
>
> Once those are taken care of, it compiles and runs (I used the heap
> allocation because it was the most expedient solution). However. On my
> very new, very expensive, very high end computer, the frame rate goes from
> 75 for the 0.11.4 release we just made, to 15. That's a phenomenal decrease
> in performance! I'll try it tomorrow on my crappy work machine running
> Ubuntu, to see how it is there. Is this kind of performance decrease
> expected?
> --
> https://code.launchpad.net/~kiithsacmp/stellarium/glexperiment/+merge/121962
> You are the owner of lp:~kiithsacmp/stellarium/glexperiment.
>