AstrOmatic forum

Full Version: WW-1.8.0 and DS9 Polygon data
You're currently viewing a stripped down version of our content. View the full version with proper formatting.
Hello,

somehow the behavior of the inclusion of DS9 polygon data changed. While in early versions (my old one was 1.3) expected the word "POLYGON" to be written in uppercase in the region files, the new version expects it to be written in lowercase.

That is, of course, no major problem, but I think, it needs to be documented, since the docs are still speaking from uppercase "POLYGON".

If that induces no other problems, it could be wise to change src/vector.c, line 127 from:
if (str2 && !(strstr(str2, "polygon")))
to:
if (str2 && !(strstr(strtolower(str2), "polygon")))
so that all possible uppercase/lowercase typos would be recognized.

Best regards Jan
Sorry jankoh,
in fact the doc is not accurate about polygons.
I'm working on a new release so new documentation will also be available.
About "POLYGON", I dropped the compatibility with SAOimage (this is in the doc) and the old versions of ds9 available to me contains only lower case.

Can you tell me what version of ds9 you are using and send it to me? Old versions of ds9 are not available on the ds9 official site.

Thanks.
Chiara.
Hi,

I'm also using a quite recent version of ds9. The thing was, that we put that stuff into our pipeline, which changed the lowercase polygon to uppercase, for whatsoever reason. As I did not write that pipeline, but just took it over and changed/modernized things, I stumbled over that problem.

No problem for me anymore, one small sed-script and things were OK again, I also changed that part of the pipeline in the way, not to generate uppercase POLYGON files.

I just wanted to let you know this, in case somebody else stumbles upon. Smile
(And maybe the docs are updated now...) :-P

Best regards Jan
Reference URL's