pinpoint-team team mailing list archive

[Andrew Straw <strawman@xxxxxxxxxx>]

Stéfan van der Walt wrote:
> Hi Andrew
> 
> This looks great!  I have published a branch at
> 
> lp:~stefanv/pinpoint/dev
> 
> that contains:
> 
> - reformatting for 80 characters line-width
> - displaying the images the right way up
> 
> I'll play with the code some more and try to fix the display problem.
> 
> Regards
> Stéfan
> 

Thanks, to you too.

Clearly we need a view option to change the display of the images. We
need at least the following: no transform, flip y, rotate 90,180,and 270
degrees. (Various camera manufacturers seem to have differing ideas
about where the origin, right, and up should be and sometimes cameras
get mounted upside down or sideways.) These are purely be for display
purposes and would not affect any underlying transforms, which should
always be specified in the raw pixel coordinates.

I've gone ahead implemented "display_transform" option and merged your
changes into my branch ( lp:~astraw/pinpoint/dev ). There's no GUI
component to switch display_transform yet, but the underlying machinery
is now under the hood. I've also made the setup.py script use setuptools
to install the "pinpoint_calibration_gui" script on one's system.

I try to keep my column width down, but clearly I'm exceeding 78
characters. You don't use emacs, do you? If so, any tips on how to keep
the linewidth down?

-Andrew