Skip to content

New package: DS9ui v1.0.0 #133313

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

UUID: 03da5ba1-8776-4371-889c-f445af0a490c
Repo: https://github.com/astrozot/DS9ui.jl.git
Tree: 2df98d1e74c53d0d0df36ab1e2aad12543d8ddfa

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
Copy link
Contributor

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. New package registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@aplavin
Copy link
Contributor

aplavin commented Jun 20, 2025

Could you please add a comparison with https://github.com/JuliaAstro/SAOImageDS9.jl? How these packages differ?

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Jun 20, 2025
@astrozot
Copy link

The package SAOImageDS9.jl implements a relatively low-level interface to SAOImage DS9, and essentially is limited to the set and get XPA commands. My package aims at developing a higher-level interface, including, for example, interactions with DS9 to measure objects' properties or extractions of masks' boundaries. Additionally, I see that SAOImageDS9.jl has not received any significant update in the last 5 years, so I am uncertain if there is any room for a significant further development of that package: an open issue regarding a proposal for further development of the SAOImageDS9 package is "frozen" to January 2021.

@aplavin
Copy link
Contributor

aplavin commented Jun 20, 2025

Can you add the functionality comparison to your readme?

Regarding

Additionally, I see that SAOImageDS9.jl has not received any significant update in the last 5 years, so I am uncertain if there is any room for a significant further development of that package: an open issue regarding a proposal for further development of the SAOImageDS9 package is "frozen" to January 2021.

You (or anyone else) can become a maintainer :) It's more convenient for everyone to have a single DS9 interaction package...

@aplavin
Copy link
Contributor

aplavin commented Jun 20, 2025

pinging SAOImageDS9 devs: @emmt @abhro @icweaver @sefffal

@astrozot
Copy link

You (or anyone else) can become a maintainer :) It's more convenient for everyone to have a single DS9 interaction package...

I see your point, and I will be happy to contribute to the SAOImageDS9 package, provided the timescale for this kind of contribution is reasonable and the maintaner(s) agree.

As a side (vaguely related) point, there are fundamental and widely used packages (StaticArrays, to name one) that have tens of unanswered open issues and tens of open pull requests. Some of these issues (or pull requests) raise (and fix) critical bugs. I know that maintaining a package requires a lot of effort, but I am not sure how to deal with these situations (in the case of StaticArrays I made a fork that fixes a critical bug I found, but does it for me only...).

@abhro
Copy link

abhro commented Jun 20, 2025

Yeah, I would highly recommend merging functionalities into the existing SAOImageDS9.jl package, where we can provide both the low-level and high-level interfaces (as is done with HDF5.jl for example)

@astrozot
Copy link

Thank you for your prompt reply @abhro , I will make a pull request to SAOImageDS9.jl then.

@icweaver
Copy link

icweaver commented Jun 20, 2025

Happy Friday all! Love to see this, thanks for sharing such a cool package @astrozot. I opened up tracking issues here:

and would be happy to help support merging things on the JuliaAstro side

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants