[Kstars-devel] Review Request: Fix dissapearing Milky Way at high zoomlevel on QPainter

[Henry de Valence]

FYI, the reason that the clipping was done manually was because there are
issues with some projections, and the fact that QPainter's clipping is
meant for flat objects, while this is doing clipping in spherical
coordinates, clipping along great-circle arcs instead of straight lines.

When I was refactoring the code into the two painting backends, I tried
just using QPainter's clipping, but there were regressions. Maybe it works
now -- but did you check all of the projections?

Cheers,
Henry de Valence

P.S. I realize it would be better if I commented on the reviewboard, but I
don't have an account, so I'm mailing here.

On Sun, Mar 18, 2012 at 03:43, Rishab Arora <ra.rishab at gmail.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104320/
>
> On March 18th, 2012, 12:11 a.m., *Rafal Kulaga* wrote:
>
> This patch indeed fixes the problem, but I wonder if the clipping here is needed at all? It seems that we can get rid of it.
>
>  I think so too! And that is what I wanted to confirm if we get any advantage at all from ignoring points manually instead of painters perform the clipping themselves. Attached p3.patch which works on both backends on the same principle.
>
>
> - Rishab
>
> On March 17th, 2012, 11:12 p.m., Rishab Arora wrote:
>   Review request for KStars, Rafal Kulaga and Akarsh Simha.
> By Rishab Arora.
>
> *Updated March 17, 2012, 11:12 p.m.*
> Description
>
> Fixes bug https://bugs.kde.org/show_bug.cgi?id=266929 for QPainter backend only.
> At high zoomlevel, the points leave the FOV and are ignored completely, whereas their fill should not be.
> Used clipping in QPainter to fix bug.
>
>   Testing
>
> Tested at various zoomlevels and resolutions. No issues found.
>
>   *Bugs: * https://bugs.kde.org/show_bug.cgi?id=266929<http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=266929>
> Diffs
>
>    - kstars/projections/projector.h (966d002)
>    - kstars/projections/projector.cpp (e8c1aff)
>    - kstars/skyqpainter.cpp (43f06e8)
>
> View Diff <http://git.reviewboard.kde.org/r/104320/diff/>
>
> _______________________________________________
> Kstars-devel mailing list
> Kstars-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kstars-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20120323/8cc69fc6/attachment.html>