Integrate ADQL-to-LSDB with TAP server: Rebase and convert DataFrame to return payload #41
+0
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Successfully integrated the TAP server with the
bin/adql_to_lsdb.pymodule to enable ADQL query execution via LSDB with DataFrame conversion to VOTable format.Changes Made
Rebased onto
dtj-adql-to-lsdbbranch - Provides access to the production-readybin/adql_to_lsdb.adql_to_lsdb()functionCherry-picked TAP server code from
copilot/create-tap-server-prototypebranch (PR Add TAP Server Prototype with ADQL Parser, LSDB Integration, and TAP_SCHEMA Support #40)Modified
tap_server/tap_server.py:bin/adql_to_lsdbmodule from parent directoryexecute_adql_query()function that:real_adql_to_lsdb.adql_to_lsdb(query)to generate Python codeexec()resultDataFrame from execution context.compute()if needed)result_df.to_dict('records')list(result_df.columns)/syncendpoint to useexecute_adql_query()instead of sample data generationUpdated
tap_server/requirements.txt:queryparser-python3,lsdb,pandas,dask[complete]Updated
tap_server/README.md:DataFrame Conversion Flow
Files Changed
tap_server/tap_server.py- Main integration with execute_adql_query() functiontap_server/requirements.txt- Added LSDB and related dependenciestap_server/README.md- Updated documentation with integration details and proper ADQL examplesbin/requirements.txt- Resolved merge conflicts to include all dependencieslsdb/busy_week_2025/hats-import-tess-lc-catalog.ipynb- Resolved merge conflictsThe implementation is complete and ready for testing with a properly configured LSDB installation.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.