D3 candlestick example error

[curtispaullithia]

Regarding using the candlestick example... https://observablehq.com/@d3/candlestick-chart

Data looks like this:
{"Date":"2017-12-08T00:00:00.000Z","Open":170.490005,"High":171,"Low":168.820007,"Close":169.369995,"Adj Close":168.034302,"Volume":23355200}

Judging by error and code (and I may be incorrect) it seems like it's expecting Date to be a number instead of a string?

Do I need to fix the example code, or the example data or am I potentially using this thing incorrectly?

[curtispaullithia]

[Fil]

In this notebook the cell that reads the data converts the date field from a string to a javascript Date, thanks to the typed: true option

aapl = FileAttachment("aapl-2.csv").csv({typed: true}).then(data => data.slice(-120))

In a typical use of D3 you might use d3.csv("url of the file", d3.autyType) instead.

[curtispaullithia]

Its not my data, it's the example data I downloaded from https://observablehq.com/@d3/candlestick-chart. The resulting downloaded JSON file shows Date as a string. Using unchanged example CandlestickChart function/code and simply passing the JSON text to it. So the data is not correct?

[curtispaullithia]

[curtispaullithia]

This is not the correct date format in the JSON data? "Date":"2017-11-17T00:00:00.000Z"

[mbostock]

Not if you’re passing it directly to CandlestickChart, no. You have to pass e.g. new Date("2017-11-17T00:00:00.000Z") instead of "2017-11-17T00:00:00.000Z". Or, like I said above, you need to use a reviver when you call JSON.parse, or otherwise convert your strings back to dates. JSON does not support dates, so you can’t use it as-is.

[curtispaullithia]

I see there was a D3.js update since this discussion. I get what you are saying.