Remove Non-functional Overlay Args from Skyview

[duytnguyendtn]

As discussed in #2976, lut, grid, and gridlabel arguments only apply for JPEG output from Skyview, though Astroquery only returns FITS objects. These values are effectively non-functional for Astroquery. This PR removes those arguments as arguments from the relevant Skyview methods and updates the docstring accordingly. Skyview tests and codestyle checks were checked and passing.

Changelog will be updated once this PR goes live and a PR number is generated.

Thanks!
Duy Nguyen (NASA HEASARC)

Fixes #2976

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.81%. Comparing base (886817e) to head (81cdbad).
Report is 94 commits behind head on main.

❗ Current head 81cdbad differs from pull request most recent head b3f3139. Consider uploading reports for the commit b3f3139 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2979      +/-   ##
==========================================
- Coverage   66.83%   66.81%   -0.02%     
==========================================
  Files         237      237              
  Lines       18327    18323       -4     
==========================================
- Hits        12248    12243       -5     
- Misses       6079     6080       +1     

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

[duytnguyendtn]

I believe the failing doctests are unrelated

[keflavich]

As noted in the referenced issue: I'm in favor of removing these w/o deprecation because they never did anything.

[duytnguyendtn]

Rebased at the request of @bsipocz! I believe the test failures are the same as in main: https://github.com/astropy/astroquery/actions/runs/8964265630/job/24664194149#step:5:640

[bsipocz]

OK, let's have this in as is without a deprecation.

If we get any user reports during the 0.4.8 dev cycle about the exception they now receive we can roll back and do it properly with a deprecation and a warning.

[bsipocz]

CI failure is unrelated and should be addressed separately.

Thank you @duytnguyendtn, and welcome on board!