[Kstars-devel] Review Request 126452: I implemented the Altitude vs. Time tool with QCustomPlot library. Now, the Zoom and Pan features are ready for use.

Jasem Mutlaq mutlaqja at ikarustech.com
Tue Dec 22 05:18:24 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126452/#review89882
-----------------------------------------------------------


Good progress. The maximum and minimum should be limited to today, i.e. you can't zoom out pass today's curve. You need to have Local Sidereal Time in the top. You need to have object names in the chart. You need to implemented daylight-->night gradient. You need to include a cross pointer when the user selects a specific Alt/Time and knows the info (just like before). You need to indicate on the curve the rise/transit/set points.

- Jasem Mutlaq


On Dec. 21, 2015, 4:10 p.m., Raphael Cojocaru wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126452/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2015, 4:10 p.m.)
> 
> 
> Review request for KStars and Jasem Mutlaq.
> 
> 
> Repository: kstars
> 
> 
> Description
> -------
> 
> Improve Altitude vs. Time tool
> 
> 
> Diffs
> -----
> 
>   kstars/tools/altvstime.cpp 6863c47 
>   kstars/tools/altvstime.ui 2cdda14 
> 
> Diff: https://git.reviewboard.kde.org/r/126452/diff/
> 
> 
> Testing
> -------
> 
> Tested with various SkyObjects. 
> I implemented the Altitude vs. Time tool with QCustomPlot library. Now, the Zoom and Pan features are ready for use.
> I had to rewrite some pieces of code for: the computing method needed for drawing the graph, the method needed for drawing the previous curves with red colour, the method needed for cleaning the graph ( removing all curves ).
> Because I changed the KPlotWidget into a QCustomPlot Widget, there are some lines of code that can't be used anymore. I marked those lines with TODO tag, in order to find a way to solve them.
> I attach here a video presentation of the result: https://www.youtube.com/watch?v=FuZEE0341sU
> 
> 
> Thanks,
> 
> Raphael Cojocaru
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20151222/560eab6d/attachment.html>


More information about the Kstars-devel mailing list