Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Commit

Permalink
Remove the curve interpolation from the time series
Browse files Browse the repository at this point in the history
I think there are too many points too close together for this to be a
good idea.
  • Loading branch information
esheehan-gsl committed Nov 10, 2023
1 parent baccd10 commit 3bdb02b
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions src/unified_graphics/static/js/component/ChartTimeSeries.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import {
area,
axisBottom,
axisLeft,
curveBumpX,
extent,
format,
line,
Expand Down Expand Up @@ -199,17 +198,14 @@ export default class ChartTimeSeries extends ChartElement {
const rangeArea = area()
.x((d) => xScale(d.initialization_time))
.y0((d) => yScale(d.min))
.y1((d) => yScale(d.max))
.curve(curveBumpX);
.y1((d) => yScale(d.max));
const interquartileRange = area()
.x((d) => xScale(d.initialization_time))
.y0((d) => yScale(d["25%"]))
.y1((d) => yScale(d["75%"]))
.curve(curveBumpX);
.y1((d) => yScale(d["75%"]));
const medianLine = line()
.x((d) => xScale(d.initialization_time))
.y((d) => yScale(d["50%"]))
.curve(curveBumpX);
.y((d) => yScale(d["50%"]));

this.#svg.attr("viewBox", `0 0 ${this.width} ${this.height}`);
this.#svg
Expand Down

0 comments on commit 3bdb02b

Please sign in to comment.