-
-
Notifications
You must be signed in to change notification settings - Fork 182
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.
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
TypeError: Cannot set property 'NaN' of undefined #114
Comments
Whoups! Debugging is welcome since I do not have a Windows machine around... |
I put a console.log above the offending line.
It seems the settings.center children nodes are NaN for some reason, could be a Windows bug? { width: NaN,
height: NaN,
marks:
{ hAxis: ' ',
vAxis: ' ',
center: ' ',
point: ' ',
rightArrow: ' ',
topArrow: ' ',
background: ' ' },
center: { x: NaN, y: NaN }, // <- Here
aRatio: 1 } |
Whoups! This is the place here the centers are set: https://github.com/IonicaBizau/node-cli-graph/blob/4f3acc270e5718ebc8b087c750b369551976df56/lib/index.js#L49-L55 |
i have the same problem |
It looks like the defaults aren't being merged with the options. |
For me it happens in the git bash. In the |
@mariuszjamro I can confirm the |
I confirm this issue under Windows 7. |
System:
The text was updated successfully, but these errors were encountered: