brownhci / WebGazer

WebGazer.js: Scalable Webcam EyeTracking Using User Interactions
https://webgazer.cs.brown.edu
Other
3.54k stars 536 forks source link

Calibration demo doesn't need plotting canvas? #209

Closed djkramnik closed 3 years ago

djkramnik commented 3 years ago

On looking into the calibration demo, I can't see the purpose of the plotting canvas. If I remove the canvas altogether, and then remove the lines of code that try to reference it, everything seems to work fine still. If its true that the canvas is currently a redundant piece of the calibration demo, perhaps remove it for clarity's sake.

djkramnik commented 3 years ago

nvm, missed the canvas ref in webgazer.js itself