Skip to content
This repository has been archived by the owner on Feb 4, 2023. It is now read-only.

Median output, fixes #1, optionally raw, fixes #2 #4

Merged
merged 2 commits into from
Nov 12, 2015

Conversation

dirkonet
Copy link
Contributor

lazy median implementation: assumes sample size to be odd, array size to be larger than half of the sample size

lazy median implementation: assumes sample size to be odd, array size to be larger than half of the sample size
returns raw value when ?raw is added to output.php call
@dirkonet dirkonet changed the title Median output, fixes #1 Median output, fixes #1, optionally raw, fixes #2 Nov 11, 2015
@dirkonet dirkonet mentioned this pull request Nov 11, 2015
@kiliankoe
Copy link
Member

awesomesauce, @sjmielke willste nochmal hochladen? 😄

kiliankoe added a commit that referenced this pull request Nov 12, 2015
Median output, fixes #1, optionally raw, fixes #2
@kiliankoe kiliankoe merged commit c86f33a into master Nov 12, 2015
sort($vals, SORT_NUMERIC);
$val = $vals[$medianElement];
if(isset($_GET["raw"])) {
echo $val;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eine Kleinigkeit: Gerade bei raw sollte doch bestenfalls nicht das medianElement genommen werden, sondern das letzte, oder?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich hab nur gemacht, was in #2 stand – letzter gemessener Wert lässt sich natürlich auch ausgeben bei Bedarf, könnte man dann über ?raw und ?median machen oder so

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, hast Recht. Hatte da ja auch vom Median gesprochen. Für raw bietet sich das wahrscheinlich doch aber besser direkt als letzter Wert an. Oder wie du meinst^^

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#5 nimmt jetzt darauf Rücksicht – ist aber nicht abwärtskompatibel. Der Median wird jetzt nur noch verwendet, wenn ?median übergeben wird.

@dirkonet dirkonet deleted the dirkonet-patch-1 branch November 12, 2015 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants