IPP Software Navigation Tools IPP Links Communication Pan-STARRS Links

Opened 20 years ago

Closed 20 years ago

Last modified 20 years ago

#711 closed defect (fixed)

psImage parent vs subimage

Reported by: eugene Owned by: David.Robbins@…
Priority: high Milestone:
Component: mathtypes Version: 0.10.0
Severity: normal Keywords:
Cc:

Description

some of the code, as well as the sdrs, are inconsistent on the issue of whether
an operation on an image refers to the subimage coordinates or the parent
coordinates. this is particularly a problem when mixing subimages of subimages
or regions and subimages. We (IfA) have decided that it makes sense for all
image operations to refer to the *parent* coordinates; thus actions are
independent of whether they operate on an image or subimage, they use the same
coordinates:

feature = psImageFindFeature (image);
region = psRegionForSquare(feature->x, feature->y, 50);
region = psRegionForImage(image, region);
subimage = psImageSubset(image, region);

result = psFeatureAnalysis(subimage, feature->x, feature->y);

I have made several fixes to functions which were needed by our tools in active
development (psphot, ppImage). there are others which need to be fixed. I have
modified (in rel10_ifa):

psImageSubset
psRegionForImage
psImageCountPixelMask

a variety of other functions likely will need to be adjusted:

psImageSet()
psImageGet()
psImageRow()
psImageCol()
psImageSlice()
psImageCut()
psImageTransform()
psImageFitPolynomial() - coeffs should refer to the parent coords
psImageEvalPolynomial() - ditto
psImagePixelInterpolate()
psPixelsToMask()

Change History (5)

comment:1 by jhoblitt, 20 years ago

blocked: 717

comment:2 by David.Robbins@…, 20 years ago

Owner: changed from gusciora@… to David.Robbins@…

comment:3 by David.Robbins@…, 20 years ago

Resolution: fixed
Status: newclosed

Fixed functions accordingly. Merged in changes where appropriate.

comment:4 by Paul Price, 20 years ago

Keywords: VERIFIED added

Bug has been resolved.... closing.

comment:5 by Paul Price, 20 years ago

Keywords: VERIFIED removed

Bugs have been fixed... closing.

Note: See TracTickets for help on using tickets.