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
Branch
update-median (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 23343
Build 23361: arc lint + arc unit
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 77065

View Options

kstars/fitsviewer/fitshistogram.cpp

Loading...