Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ECDF Stat #832

Merged
merged 17 commits into from
Aug 2, 2023
Merged

ECDF Stat #832

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
be14e42
Basic version of the eCDF stat.
ASmirnov-HORIS Jul 26, 2023
d53551a
Refactor ECDFStat.
ASmirnov-HORIS Jul 27, 2023
c66b8ef
Add an interpolation to ECDF.
ASmirnov-HORIS Jul 27, 2023
e2eb316
Add autotests for the ECDFStat.
ASmirnov-HORIS Jul 27, 2023
5411aa4
Fix ECDFStat (case when in one group there is only NaN's).
ASmirnov-HORIS Jul 27, 2023
4159166
Add 'pad' parameter to the StepGeom (for the eCDF).
ASmirnov-HORIS Jul 27, 2023
9d017f5
Add stat_ecdf() function. Tiny fixes in other stat functions.
ASmirnov-HORIS Jul 27, 2023
49a8b21
Specify limits for the stat_ecdf() function.
ASmirnov-HORIS Jul 28, 2023
5447c08
Add docstrings for stat_ecdf().
ASmirnov-HORIS Jul 28, 2023
2fc1c01
Small fixes in StepGeom::getPads because of sampling.
ASmirnov-HORIS Jul 28, 2023
410b7d6
Move scaling of the Y axis inside the step geometry.
ASmirnov-HORIS Jul 31, 2023
cf1f5f1
Add demo notebook for the stat_ecdf().
ASmirnov-HORIS Jul 31, 2023
0c6a5b4
Mention stat_ecdf() in the future_changes.md file.
ASmirnov-HORIS Jul 31, 2023
8eb5f53
Merge branch 'master' into ecdf
ASmirnov-HORIS Aug 1, 2023
818cd96
Small refactor in the StepGeom.
ASmirnov-HORIS Aug 1, 2023
5ff48c2
Add pads to the ECDFStat.
ASmirnov-HORIS Aug 2, 2023
dd3cdd0
Add another one demo case for the eCDF.
ASmirnov-HORIS Aug 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix ECDFStat (case when in one group there is only NaN's).
  • Loading branch information
@ASmirnov-HORIS
ASmirnov-HORIS committed Jul 27, 2023
commit 5411aa4555250b939780b699a3233e6193570986
25 changes: 19 additions & 6 deletions plot-base/src/commonMain/kotlin/org/jetbrains/letsPlot/core/plot/base/stat/ECDFStat.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,23 @@ class ECDFStat(
return withEmptyStatValues()
}

val xs = data.getNumeric(TransformVar.X)
val statData = buildStat(data.getNumeric(TransformVar.X))

return DataFrame.Builder()
.putNumeric(Stats.X, statData.getValue(Stats.X))
.putNumeric(Stats.Y, statData.getValue(Stats.Y))
.build()
}

private fun buildStat(
xs: List<Double?>
): Map<DataFrame.Variable, List<Double>> {
val xValues = xs.filter { it?.isFinite() ?: false }.map { it!! }
if (xValues.isEmpty()) {
return withEmptyStatValues()
return mapOf(
Stats.X to emptyList(),
Stats.Y to emptyList(),
)
}

val ecdf: (Double) -> Double = { t -> xValues.count { x -> x <= t }.toDouble() / xValues.size }
Expand All @@ -37,10 +50,10 @@ class ECDFStat(
}
val statY = statX.map { ecdf(it) }

return DataFrame.Builder()
.putNumeric(Stats.X, statX)
.putNumeric(Stats.Y, statY)
.build()
return mapOf(
Stats.X to statX,
Stats.Y to statY,
)
}

private fun linspace(start: Double, stop: Double, num: Int): List<Double> {
Expand Down