Differential D27880

Update median calculation
ClosedPublic
Actions

Authored by murveit on Mar 6 2020, 8:27 AM.

Details

Reviewers
mutlaqja
Commits
R321:acf0f6eee078: Update median calculation
Summary

Simplifies the median calculation, and defends against edge cases that can cause crashes.

Test Plan

Should compute the same results as the previous median, and should not crash.

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
murveit created this revision.Mar 6 2020, 8:27 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 6 2020, 8:27 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
murveit requested review of this revision.Mar 6 2020, 8:27 AM
mutlaqja accepted this revision.Mar 6 2020, 8:29 AM
This revision is now accepted and ready to land.Mar 6 2020, 8:29 AM
Closed by commit R321:acf0f6eee078: Update median calculation (authored by murveit, committed by mutlaqja). · Explain WhyMar 6 2020, 8:30 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkstars/fitsviewer/fitshistogram.cpp (83 lines)

Diff 77067

View Options

kstars/fitsviewer/fitshistogram.cpp

Loading...