Opened 9 years ago

Last modified 8 years ago

#1024 closed defect

Petascope ingestion of non-geo-referenced images — at Version 8

Reported by: Alex Dumitru Owned by: Vlad Merticariu
Priority: major Milestone: 9.3
Component: petascope Version: development
Keywords: Cc:
Complexity: Medium

Description (last modified by Alex Dumitru)

When importing this GML (http://pastebin.com/RfhmM4T1) I get a coverage that starts at (1,1), although the origin is set at (0.5, 0.5). Is this correct behavior? I am assuming some rounding is performed somewhere for Index2D CRSes.

Change History (9)

comment:1 by Vlad Merticariu, 9 years ago

Can you attach the coverage description after ingestion?

comment:2 by Bang Pham Huu, 9 years ago

I think the reason is based on this function and also want to know why need to do this? Please answer if you have time.

  def get_origin(self):
	        """
	        Returns the origin of the dataset. This is calculated using the origin from the dataset + 0.5 of an offset
	                vector, as petascope requires it so.
	        :rtype: list[str]
	        """
	        geo = self.gdal_dataset.GetGeoTransform()
	        # Add 0.5 * size of pixel to make sure there's no error in petascope
	        # Should be removed once petascope handles numeric computations correctly
	        return str(geo[0] + 0.5 * self.gdal_dataset.GetGeoTransform()[1]), \
	               str(geo[3] + 0.5 * self.gdal_dataset.GetGeoTransform()[5])

comment:3 by Alex Dumitru, 9 years ago

@Bang: The origin in petascope is considered to be in the middle of the geopixel, that is top-left corner + 0.5 offset vector. The whole geo-model is described here http://www.rasdaman.org/wiki/PetascopeUserGuide
The comment delimited by # is not correct, only the one between """ """ is.

This is a bug in petascope.wcst, not in WCSTImport, the created coverage is correct.

by Bang Pham Huu, 9 years ago

Attachment: afteringestion.gml added

comment:4 by Bang Pham Huu, 9 years ago

Thanks Alex for your time I will read the article, I also would like to add the file as Vlad requested.

comment:5 by Dimitar Misev, 9 years ago

Milestone: 10.0

comment:6 by Dimitar Misev, 9 years ago

Owner: changed from Vlad Merticariu to Bang Pham Huu
Status: newassigned

comment:7 by Alex Dumitru, 9 years ago

Description: modified (diff)
Owner: changed from Bang Pham Huu to Vlad Merticariu

comment:8 by Alex Dumitru, 9 years ago

Description: modified (diff)

Please leave it assigned to Vlad. This is more of a standards decision than actual implementation.
@Vlad Should the origin be in the middle of the pixel for non-georef grids as well?

Note: See TracTickets for help on using tickets.