Differential D14149

calculating cumulative frequency corrected
ClosedPublic
Actions

Authored by wreissenberger on Jul 16 2018, 6:29 AM.

Details

Reviewers
mutlaqja
Commits
R321:7804e618ed41: calculating cumulative frequency corrected
Summary

The current implementation of the cumulative frequency is implemented very inefficient. I changed it making its runtime linear.

Additionally, I changed the method for changing an element inside a QVector. At least on my raspberry, this seems to have side effects and changed the "channels" attribute leading to a 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.
wreissenberger created this revision.Jul 16 2018, 6:29 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptJul 16 2018, 6:29 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
wreissenberger requested review of this revision.Jul 16 2018, 6:29 AM
mutlaqja accepted this revision.Jul 22 2018, 6:44 AM
This revision is now accepted and ready to land.Jul 22 2018, 6:44 AM
Closed by commit R321:7804e618ed41: calculating cumulative frequency corrected (authored by mutlaqja). · Explain WhyJul 22 2018, 6:45 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

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

Diff 38193

View Options

kstars/fitsviewer/fitshistogram.cpp

Loading...