-
Notifications
You must be signed in to change notification settings - Fork 11
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
PeasyCam Broken with p5.js v0.7.2 September 02, 2018 #5
Comments
Also, could you indicated how to initialise the view with the Z going up towards the top of the browser? |
Getting the same issue from some users on OpenProcessing. Not sure hot to fix, but must a variable naming issue. |
While we await the above PR approval, OpenProcessing users who want to use 0.7.2 with EasyCam can drop the following fix into setup() prior to invoking createEasyCam() // fix for EasyCam to work with p5 v0.7.2
Dw.EasyCam.prototype.apply = function(n) {
var o = this.cam;
n = n || o.renderer,
n && (this.camEYE = this.getPosition(this.camEYE), this.camLAT = this.getCenter(this.camLAT), this.camRUP = this.getUpVector(this.camRUP), n._curCamera.camera(this.camEYE[0], this.camEYE[1], this.camEYE[2], this.camLAT[0], this.camLAT[1], this.camLAT[2], this.camRUP[0], this.camRUP[1], this.camRUP[2]))
}; For example: http://www.openprocessing.org/sketch/621401 |
hi, i'm still having problem working with newer versions... please help.............. |
The fix I have posted above also works with p5 v0.7.3 |
hi again... thanks for your response... your code works just fine... i'm trying to continue working on it... thanks again |
why do i get this warning? |
Off topic: |
thanks a lot! after some modifications suggested by you, i got it working... and it seems mySketch.js should be loaded in body tag... don't know why, but it is working now... |
The fix I posted above also works with p5 v0.9.0 |
Peasy cam doesn't load with the simple example provided in the readme - any ideas?
The text was updated successfully, but these errors were encountered: