Small bug in calculating mullion offset

TACC / MassivePixelEnvironment

Massive Pixel Environment (MPE) is a Processing library for easily extending sketches to distributed display environments.

Other

53 stars 10 forks source link

Small bug in calculating mullion offset #10

Closed isaacliao closed 11 years ago

isaacliao commented 11 years ago

On line 238 of Configuration.java,

offsets_[1] = (minj)tileRes[1] + minibezels[1];

should be:

offsets_[1] = (minj)tileRes[1] + minjbezels[1];

I was wondering why changing mullionHeight in configuration.xml didn't seem to be working correctly, causing tiles to become vertically offset from each other across vertical boundaries. Making this change fixed it.

Otherwise the package works great! Thanks for making it!

Isaac Liao VIDI Lab UC Davis

bmwesting commented 11 years ago

Hi Isaac,

Thanks for tracking down the issue. I'll update the code to perform bezel compensation correctly and issue a new version.

Thanks, Brandt

bmwesting commented 11 years ago

Fixed in commit b80f4c4ae0c03769ec6a86f264b7b05747396b87. Updated binary to follow by 5PM CST.