Skip to content
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

difference on Kinect.Image between v5.1 and v4.5 #921

Open
dbsima opened this issue May 8, 2014 · 9 comments
Open

difference on Kinect.Image between v5.1 and v4.5 #921

dbsima opened this issue May 8, 2014 · 9 comments

Comments

@dbsima
Copy link

dbsima commented May 8, 2014

With v4.5 the loaded images look great http://jsfiddle.net/dbsima/5d2M3/ but with v.5.1 http://jsfiddle.net/5d2M3/1/ the images are losing pixels. What am I doing wrong?

@lavrton
Copy link
Contributor

lavrton commented May 8, 2014

@pgrodrigues
Copy link

Hey dbsima,

Some methods were changed in v5.0.0. For example:
.getX() is now .x()
.setSize(100, 100) is now .size({width: 100, height: 100})

I changed your jsfiddle, and it seems to be working now
http://jsfiddle.net/5d2M3/5/

@dbsima
Copy link
Author

dbsima commented May 8, 2014

@pgrodrigues I knew about those changes. My problem is that the images with v5.1 don't have de pixel-ratio as with v4.5.

@lavrton
Copy link
Contributor

lavrton commented May 15, 2014

Can you provide more information about "losing pixel"? Screenshots?

@dbsima
Copy link
Author

dbsima commented May 15, 2014

@lavrton just look at the 2 jsfinddles http://jsfiddle.net/dbsima/5d2M3/ and http://jsfiddle.net/5d2M3/1/ the images in the first one look better.

@lavrton
Copy link
Contributor

lavrton commented May 15, 2014

I have ubuntu with chrome 34 and I don't see any difference. What is your system?

@dbsima
Copy link
Author

dbsima commented May 18, 2014

I think that the difference is because of the retina display. I checked on PC with no retina display and there's no difference.

@lavrton
Copy link
Contributor

lavrton commented May 18, 2014

I think it is related to #819 and #750.

@dbsima
Copy link
Author

dbsima commented May 19, 2014

Yeah. Same problem. Sorry I didn't saw them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants