Eero Vaher

Results 271 comments of Eero Vaher

> Do you recall any reasons why the `async_to_sync` decorator wasn't used? There already is an issue about this: #2100.

I noticed that `astroquery` defines a `MaxResultsWarning`: https://github.com/astropy/astroquery/blob/c29328b32832a8fb53371e35ec6d1cb6b3a03470/astroquery/exceptions.py#L93-L97 This is more appropriate for letting the users know their row limit might be too small, so I've updated the pull request.

Does this pull request need a change log entry or has the `no-changelog-entry-needed` label been forgotten?

I have added a section to the opening post that explains the motivation for the changes this pull request proposes.

I had to rebase to solve a conflict in the change log.

I had to rebase again to resolve another merge conflict in the change log. I also decided to add a `_row_limit` property that works analogously to the other two I...

I have rebased the commits to update the change log entry.

For some reason none of the tests ran at all. Should I rebase again to start the tests?

I've updated the pull request so that the documentation about the Simbad subpackage is kept up to date regarding the configuration, and I had to rebase to resolve a merge...

I had to rebase to fix a merge conflict in the change log, again.