Skip to content

Conversation

nmearl
Copy link
Contributor

@nmearl nmearl commented Feb 10, 2020

This PR adds the ability to set the ROW_LIMIT class attribute on the Gaia instance in order to limit the number of rows selected in the submitted query.

@astropy-bot astropy-bot bot added the gaia label Feb 10, 2020
@codecov
Copy link

codecov bot commented Feb 10, 2020

Codecov Report

Merging #1641 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1641      +/-   ##
==========================================
- Coverage      63%   62.97%   -0.03%     
==========================================
  Files         185      185              
  Lines       14983    14985       +2     
==========================================
- Hits         9440     9437       -3     
- Misses       5543     5548       +5
Impacted Files Coverage Δ
astroquery/gaia/core.py 83.07% <100%> (+0.13%) ⬆️
astroquery/gaia/__init__.py 100% <100%> (ø) ⬆️
astroquery/utils/tap/taputils.py 69.44% <0%> (-6.95%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 617457b...20393b8. Read the comment docs.

@bsipocz bsipocz added this to the v0.4.1 milestone Feb 14, 2020
j = Gaia.cone_search_async(coord, radius)
r = j.get_results()

assert len(r) == 1188
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose these values will change with new DRs, but let's cross that bridge when we get there.

@bsipocz bsipocz merged commit 332a908 into astropy:master Feb 14, 2020
@bsipocz
Copy link
Member

bsipocz commented Feb 14, 2020

Thanks @nmearl!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants