<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107906/">http://git.reviewboard.kde.org/r/107906/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi Timothy, awesome work. But there are a few severe issues with this patch:
- I can't find any notice on licenses or copyrights on the vendian.org page. Maybe you can? We need to know whether this work is available in the public domain or whether the author demands attribution (CC by ...). He also needs to be fine with commercial usage. So ideally his work would be provided under the public domain or CC by SA 3.0. Since I don't find anything like that I just assume the worst and consider this work "All rights reserved" which prevents us from shipping it. So we need a nice e-mail from him stating the terms of use. Otherwise it's a show-stopper.
- The current appearance of the stars with the color values is quite clownesque. The reason is that point-like stars have a color appearance that is different from the appearance of discs (which we use to get the huge brightness range across). This issue would also have existed with my colors to some degree (although I chose the colors in a way that makes the colors less apparent and hence more realistic).
In order to "mellow" these clownesque colors and in order to make the stars look a bit more natural the next step would be to use pixmaps and colorize those. I did this as part of the preparation for task
http://www.google-melange.com/gci/task/view/google/gci2012/8122209
See
http://devel-home.kde.org/~tackat/starmockup.png
for a mockup with proper sizes and proper color shades (take the current state as a comparison: http://devel-home.kde.org/~tackat/starnow.png )
This colorization is only applied to the glow / newton spikes behind while the star-pinpoint center just has shades of white. In total this provides a more natural color appearance (for non-point depiction. I don't doubt that the vendian.org one works okish for point depictions).
Since your approach currently allows for an almost infinite range of colors (which a normal user won't be able to distinguish anyways) I'm not sure how to colorize the glow inside the pixmaps efficiently.
Due to copyright issue I'm not quite sure how to further deal with this task. Nevertheless totally awesome work but we need to resolve these issues somehow :-/
</pre>
<br />
<p>- Torsten</p>
<br />
<p>On December 25th, 2012, 5:52 a.m., Timothy Lanzi wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Marble.</div>
<div>By Timothy Lanzi.</div>
<p style="color: grey;"><i>Updated Dec. 25, 2012, 5:52 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Colors added to stars according to their spectrum description given in the catalog.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Compiled and ran, checked to see if it worked.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>data/stars/stars.dat <span style="color: grey">(40def72)</span></li>
<li>src/plugins/render/stars/StarsPlugin.h <span style="color: grey">(d979629)</span></li>
<li>src/plugins/render/stars/StarsPlugin.cpp <span style="color: grey">(4506fd4)</span></li>
<li>tools/stars/starcolor.dat <span style="color: grey">(PRE-CREATION)</span></li>
<li>tools/stars/stars.cpp <span style="color: grey">(5cdd997)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107906/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>