<table><tr><td style="">TallFurryMan created this revision.<br />TallFurryMan added reviewers: mutlaqja, wreissenberger.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />TallFurryMan requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15492">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Used system locale to display all numbers in Capture module.<br />
Made sequence file XML serialization always use the C locale for portability.<br />
Loaded whatever value is available for guide deviation, enabled or not.<br />
Loaded whatever HFR value is if file, and reset it if invalid.<br />
Loaded whatever refocus period is in the file, or reset it if invalid.<br />
Loaded whatever RA value is in file for meridian flip.<br />
Displayed proper number of decimals in the job table (based on min exposure time).<br />
Added a warning about HFR-based autofocus serialization.<br />
Fixes selection of current CCD when a sequence file is loaded.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Test plan is to be done at least in:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">a locale that uses "." as decimal point, such as english</li>
<li class="remarkup-list-item">a locale that uses "," as decimal point, such as french</li>
</ul>

<p>The differential is expected to work with variations of decimal points, but will not change how timestamps are displayed.</p>

<p>Load test vector "decimal_values.esq" from folder "Tests/capture" in the Capture module.</p>

<p>Check the exposure field box.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">With or without the change, it will display proper decimal separator, and convert to a double value without issue.</li>
<li class="remarkup-list-item">Note the thousands separator is not displayed there.</li>
</ul>

<p>Check the exposure column in the table.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Without the change, it will display as an integer or with "." as decimal separator.</li>
<li class="remarkup-list-item">With the change, decimal separator relevant to the current locale will be used.</li>
<li class="remarkup-list-item">Also note the thousands separator properly displaying if the locale requests it.</li>
</ul>

<p>Execute the job, check that the messages properly display the locale-related number format.</p>

<p>View the test vector in a text editor, and observe the bottom left checkbox settings:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Check that guiding deviation is equal to the double value seen in the file, 2 digits after decimal separator.</li>
<li class="remarkup-list-item">Check that autofocus HFR is equal to the double value seen in the file, 3 digits after decimal separator.</li>
<li class="remarkup-list-item">Check that refocus periodicity is equal to the integer value seen in the file, no decimal separator.</li>
<li class="remarkup-list-item">Check that RA hours for meridian flip is equal to the double value seen in the file, 2 digits after decimal separator.</li>
</ul>

<p>Change the values, checkbox states and jobs in the Capture module (don't do too much at once).<br />
Save the resulting sequence under another name.<br />
Check that all the resulting sequence is using C-language numbers (decimal separator ".", no thousand separator).<br />
Diff against the original test vector, check that only actual changes went in the new sequence file.</p>

<p>About the bottom left checkbox settings, check that values are always loaded from the file, even if the checkbox is not enabled.<br />
Without the change, values from a previous sequence could be retained, leading to unpredictable results.</p>

<p>Depending on the state of option "Save Sequence HFR to File", check that HFR is saved to the sequence and loaded from it, or reset to zero.<br />
Observe the warning if "Save Sequence HFR to File" is disabled and HFR-based autofocus is enabled.</p>

<p>Finally, if you have multiple CCD cameras, prepare a sequence file for each, whatever the content.<br />
Check that each of your sequences is properly selecting the correct CCD, and properly initializes frame information.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>improve__support_decimal_locales (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15492">https://phabricator.kde.org/D15492</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>Tests/capture/decimal_values.esq<br />
kstars/ekos/capture/capture.cpp</div></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>