Skip to content
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

Update tap.rst #2985

Merged
merged 1 commit into from May 3, 2024
Merged

Update tap.rst #2985

merged 1 commit into from May 3, 2024

Conversation

richardgmcmahon
Copy link
Contributor

small change to docs in 3. Using TAP+ to connect other TAP services

unsupported
print(table.get_name())

corrected to:

print(table.get_qualified_name())

changed 

unsupported 
print(table.get_name())

to:

print(table.get_qualified_name())
@bsipocz
Copy link
Member

bsipocz commented May 3, 2024

Thanks @richardgmcmahon!

@bsipocz bsipocz merged commit 2f87333 into astropy:main May 3, 2024
8 of 9 checks passed
@bsipocz
Copy link
Member

bsipocz commented May 3, 2024

Small comment: while we don't make TapPlus a private functionality its purpose is to serve the ESA modules in astroquery only and we recommend using the TAP functionalities from the pyvo package instead. Long term we plan to consolidate the two implementations into one and deprecate and remove astroquery.utils.tap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants