blurymind / 1dollar_gesture_recogniser Goto Github PK

Hi, there is a bug in RotateBy (+ there again) function.

Currently, it is

var qx = (points[i].x - c.x) * mcos - (points[i].x - c.x) * msin + c.x
var qy = (points[i].x - c.x) * msin + (points[i].x - c.x) * mcos + c.x

but it should be

var qx = (points[i].x - c.x) * mcos - (points[i].y - c.y) * msin + c.x
var qy = (points[i].x - c.x) * msin + (points[i].y - c.y) * mcos + c.y

(.y instead of .x in second half)

implementation for example: https://depts.washington.edu/acelab/proj/dollar/dollar.js

Maybe thats the reason why RotateBy is commented out when building Unistroke? (or you need this to find difference between vertical and horizontal line?)

Sorry for the delay, just checking the assetlib moderation queue now (need to program email notifications when new edits are done :)).

The assetlib unzips the contents of your repo directly in the user's project, so you should provide the addons/<youraddon>/ folder directly in this git repo, so that it's unzipped in the right location out of the box.