Skip to content

Update tap.rst#2985

Merged
bsipocz merged 1 commit into
astropy:mainfrom
richardgmcmahon:rgm-1
May 3, 2024
Merged

Update tap.rst#2985
bsipocz merged 1 commit into
astropy:mainfrom
richardgmcmahon:rgm-1

Conversation

@richardgmcmahon

Copy link
Copy Markdown
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

bsipocz commented May 3, 2024

Copy link
Copy Markdown
Member

Thanks @richardgmcmahon!

@bsipocz bsipocz merged commit 2f87333 into astropy:main May 3, 2024
@bsipocz

bsipocz commented May 3, 2024

Copy link
Copy Markdown
Member

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.

2 participants