Page MenuHomePhabricator

Date data input separator "." no longer parsed correctly
Closed, DeclinedPublic

Assigned To
None
Authored By
Pietrasagh
Sep 12 2020, 11:40 AM
Referenced Files
F32280717: soll.png
Sep 12 2020, 11:40 AM
F32280730: ist_data.png
Sep 12 2020, 11:40 AM
F32280723: ist.png
Sep 12 2020, 11:40 AM

Description

Date data input separator "." is not longer parsed correctly. Bug appeared some time ago. I first nottied it on pl.wikipedia.org (1 -2 weeks ago), now it's also on en.wikipedia.org, Mediawiki and Vega 2 editor. When graph is generated by Graphoid date is parsed correctly and graph is generated as expected, please see my sandbox.

Existing output

ist.png (225×240 px, 2 KB)

debug from Vega 2 editor shows that d3 parse of input is null
ist_data.png (199×184 px, 5 KB)

Expected output
soll.png (239×252 px, 5 KB)

Vega 2 JSON for example above

{
  "width": 200,
  "height": 200,
  "scales": [
    {
      "type": "time",
      "name": "x",
      "domain": {"data": "chart","field": "x"},
      "range": "width",
      "zero": false
    },
    {
      "type": "linear",
      "name": "y",
      "domain": {"data": "chart","field": "y"},
      "zero": false,
      "range": "height",
      "nice": true
    }
  ],
  "marks": [
    {
      "type": "line",
      "properties": {
        "enter": {
          "y": {"scale": "y","field": "y"},
          "x": {"scale": "x","field": "x"},
          "stroke": {"value": "black"}
        }
      },
      "from": {"data": "chart"}
    }
  ],
  "axes": [{"type": "x","scale": "x"},{"type": "y","scale": "y"}],
  "data": [
    {
      "format": {"parse": {"y": "integer","x": "date"},"type": "json"},
      "name": "chart",
      "values": [
        {"y": 1,"series": "y","x": "1.10.2020"},
        {"y": 2,"series": "y","x": "2.10.2020"},
        {"y": 3,"series": "y","x": "3.10.2020"}
      ]
    }
  ]
}

Event Timeline

Pietrasagh triaged this task as Lowest priority.

It seems to be browser related ("." works in chrome but not in firefox). Maybe graphoid use same libs as chrome/ium
See Vega issue