Skip to content

Remove imageanalysis parser for CRTF imports #1219

@pford

Description

@pford

Due to performance and incompatibility issues, remove use of casa imageanalysis CRTF import implementation and use CARTA implementation only. This will improve the CRTF performance issue noted in #1216 .

With the addition of annotation regions in CARTA in v4.0, some regions which are not supported in CRTF (compass and ruler) are exported with a preceding # so that the CRTF file remains supported by other packages such as casaviewer, astropy, etc. The # usually indicates a comment line and is skipped by the casa CRTF parser. Since these lines must be checked in CARTA for an annotation region definition, we can no longer use imageanalysis to import CRTF files since there will be no indication that these regions were skipped.

The imagenanalysis implementation has slower performance because, as part of the import to create casa annotation regions, (1) pixel coordinates are converted to world coordinates (direction coordinate only, else fails) and (2) a composite region of all regions in the file is created. However, these features are not required or used by CARTA. In addition, any import failure in the file causes the entire import to fail, which was one motivation for the CARTA parser implementation as a workaround.

For now, imagenalysis continues to be used for CRTF region export for formatting, particularly world coordinate strings.

Metadata

Metadata

Assignees

Labels

casaIssue due to casacore implementationperformance

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions