astronomy-commons / lsdb

Large Survey DataBase
https://lsdb.io
BSD 3-Clause "New" or "Revised" License
19 stars 5 forks source link

added kwargs on hats.read_hats to pass http headers #485

Closed Schwarzam closed 2 weeks ago

Schwarzam commented 2 weeks ago

Added kwargs to hats.read_hats for HTTP headers support

This is related to https://github.com/astronomy-commons/hats/pull/411

And issue https://github.com/astronomy-commons/hats/issues/410

Even without the pull request in the hats repo, the code in this pull request works.

This code functions as expected, even without the corresponding pull request in the hats repository. Tested with and without headers.

github-actions[bot] commented 2 weeks ago
Before [5fbe2e22] After [4c5fe021] Ratio Benchmark (Parameter)
30.1±0.6ms 30.5±0.6ms 1.01 benchmarks.time_polygon_search
14.5±0.02s 14.4±0.03s 0.99 benchmarks.time_create_large_catalog
2.18±0.02s 2.15±0.01s 0.99 benchmarks.time_create_midsize_catalog
102±0.8ms 101±0.6ms 0.99 benchmarks.time_kdtree_crossmatch
16.9±2ms 15.9±0.4ms 0.94 benchmarks.time_box_filter_on_partition

Click here to view all benchmarks.