Fixes for SAMP migration from astropy to pyvo

[pllim]

This is a follow-up of #729

Related issue: #761

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.88%. Comparing base (69c3a1f) to head (cda1f62).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #762   +/-   ##
=======================================
  Coverage   79.88%   79.88%           
=======================================
  Files          91       91           
  Lines       10312    10312           
=======================================
  Hits         8238     8238           
  Misses       2074     2074           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pllim]

I will add a change log if you think we need it.

@astrofrog also suggested that pyvo removes the samp_hub CLI completely, so if you want to go with that instead, then this PR needs to be modified.

[astrofrog]

Just to expand on my suggestion, we could move to a model where we no longer have a CLI command but set it up so that users can instead do e.g.:

python -m pyvo.samp.hub

[bsipocz]

Thank you @pllim. I'll see to the changelog before doing a release.

[bsipocz]

🤦‍♀️ of not noticing this in review.

[pllim]

It happens. No worries.

[bsipocz]

@tomdonaldson @fjebaker - what do you think about the cli suggestion above?

[fjebaker]

I think it's a good idea! I think it fits in naturally with other Python module tools like python3 -m http.server.

P.S.: I am still planning to submit some SAMP patches, but unfortunately after my migration PR got merged I've struggled to find the time to work on them... :(

[bsipocz]

P.S.: I am still planning to submit some SAMP patches, but unfortunately after my migration PR got merged I've struggled to find the time to work on them... :(

No worries, you get to it when you get to it, any attention to samp on any timescale will be better than we had for it in the past many years

[bsipocz]

I think it's a good idea! I think it fits in naturally with other Python module tools like python3 -m http.server.

OK. If the other maintainers are OK with it, then I can tag a new 1.9.1 release once this change is done.

[pllim]

I opened #763