Lynga loader read for Pull Request

[casazza]

Lynga Open Cluster Catalog loader.
Unit tests needed change to allow for 1 object at a Dec or exactly 0.0

[mrosseel]

looks good at first glance.
The location in the menu should be changed I think, should be where the Harris catalog is, it seems it's now in the top menu (maybe for testing).
If you keep on producing catalogs we'll probably need to subdivide the menus a bit further into open clusters, globulars, etc. We'll cross that bridge when we get there :).

[casazza]

@mrosseel I added it in alphabetical order in the DSOs list. That feels most correct as everything else was in alphabetic order. Are you seeing something different?

I do agree that we are getting a number of catalogs and should revisit the order.

[brickbots]

Fantastic @casazza ! Thank you again for another great catalog addition. Did you happen to have a bead on the usage/license for this? I didn't see this in the readme and I tried to check the source URL and it looks like the server is down.

[casazza]

@brickbots This was free to use. No special notification or attribution required. I added it to the Catalog's doc. That should be more than sufficient.

[mrosseel]

there are already common facilities to find if something is in our database called ObjectFinder, so maybe you can use those for this use case? Or extend them / add a utility function if you have a unique use case

[mrosseel]

just have a look at that one remark I gave but for the rest it looks good, haven't tested it locally yet though.