From: Daniel M G. <dm...@uv...> - 2008-12-29 12:39:09
|
Hi Everybody, I just committed this change. 2008-12-29 dmg <dm...@uv...> * panorama.h, math.c, adjust.c, queryfeature.c, filter.h, parser.c: Added panini projection. the credit goes to Thomas Sharpless who rediscovered it, and to Bruno Postle, who did the original math. I'll try to add the "architectural" in the next few days. I'll let Bruno and Tom provide the background and math, both are fascinating. I have only tested it going from equirectangular to panini, but it should work like any other. We might need some more changes to the library before hugin can use it, though. --dmg |
From: Daniel M G. <dm...@uv...> - 2008-12-29 22:22:27
|
>> I have only tested it going from equirectangular to panini, but it >> should work like any other. We might need some more changes to the >> library before hugin can use it, though. Yuval> I found a small bug in the library (even before the introduction of Yuval> Panini) that would enumerate one projection too many in the hugin drop Mmm, I just checked and I think libpano code is ok. It might be the code in hugin. panoProjectionFormatCount returns exactly the enumber of projections available (currently 13, including panini), numbered from 0 to 12. Panini is #12 in panotools. Anybody wants to try it? Yuval> down of available projections. And Panini was missing in there, so I added Yuval> it. I don't think I have commit rights to panotools, and I'm anyway on Yuval> very limited infrastructure for another ten days. Find my patch to libpano Yuval> attached. The hugin stuff will come in a subsequent mail, with a blog Yuval> article on how to do that (I hope to entice newbie contributors to put Yuval> their hands under the hood of hugin, and maybe you will help me with Yuval> something similar for the projections in panotools?). I had done the changes last night already, and I committed them few minutes ago. your code was good, except that it was missing the maximum vFOV and hFOV values that libpano should return to hugin. By the way, I have hardcoded 220 degrees as the maximum hFOV. Anything bigger is just, well, unusable ;) vFOV is set to 180, but 90 is a more practical one. --dmg -- Daniel M. German "We die. That may be the meaning of life. But we do language. That may be Toni Morrison -> the measure of our lives." http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |
From: Daniel M G. <dm...@uv...> - 2008-12-29 22:22:28
|
>> I have only tested it going from equirectangular to panini, but it >> should work like any other. We might need some more changes to the >> library before hugin can use it, though. Yuval> I found a small bug in the library (even before the introduction of Yuval> Panini) that would enumerate one projection too many in the hugin drop Mmm, I just checked and I think libpano code is ok. It might be the code in hugin. panoProjectionFormatCount returns exactly the enumber of projections available (currently 13, including panini), numbered from 0 to 12. Panini is #12 in panotools. Anybody wants to try it? Yuval> down of available projections. And Panini was missing in there, so I added Yuval> it. I don't think I have commit rights to panotools, and I'm anyway on Yuval> very limited infrastructure for another ten days. Find my patch to libpano Yuval> attached. The hugin stuff will come in a subsequent mail, with a blog Yuval> article on how to do that (I hope to entice newbie contributors to put Yuval> their hands under the hood of hugin, and maybe you will help me with Yuval> something similar for the projections in panotools?). I had done the changes last night already, and I committed them few minutes ago. your code was good, except that it was missing the maximum vFOV and hFOV values that libpano should return to hugin. By the way, I have hardcoded 220 degrees as the maximum hFOV. Anything bigger is just, well, unusable ;) vFOV is set to 180, but 90 is a more practical one. --dmg -- Daniel M. German "We die. That may be the meaning of life. But we do language. That may be Toni Morrison -> the measure of our lives." http://turingmachine.org/ http://silvernegative.com/ dmg (at) uvic (dot) ca replace (at) with @ and (dot) with . |