Besancon - added reader #13
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dependencies...)
Contributor
Author
|
I don't understand why this can't be automatically merged; I'll wait for feedback before trying to merge it. |
Member
|
I think it's because it contains commits that have already been merged (e.g. 'added progressbar code'?). Try rebasing and you'll see it will raise errors. |
dependencies...)
Contributor
Author
|
I think I messed up the rebase. If I try to rebase now, I get multiple copies of each commit. I'll need to do some research to figure out how to fix it. |
astrofrog
pushed a commit
to astrofrog/astroquery
that referenced
this pull request
Jun 20, 2013
Fix scripts/README.rst bug
bsipocz
pushed a commit
to bsipocz/astroquery
that referenced
this pull request
Nov 8, 2016
Give cache directory path in HITRAN docs
keflavich
added a commit
to eas342/astroquery
that referenced
this pull request
Mar 25, 2025
# This is the 1st commit message: I made a lot of tests using AI help - I think it's all right, but I want to review it on github & see what coverage bot says # This is the commit message astropy#2: consolidated tests & expanded to cover `head_safe` cases # This is the commit message astropy#3: remove some cruft # This is the commit message astropy#4: cleanup: remove unused file # This is the commit message astropy#5: some cleanup of flake8 - but the AI choked on the easy task, so I'm not sure things are right any more... # This is the commit message astropy#6: cover the zero-size initial file case # This is the commit message astropy#7: a lot of refactoring because some tests were failing, there were some redundant checks, and there seems to be no reason to return None # This is the commit message astropy#8: compress test parameters and add `cache` to increase coverage # This is the commit message astropy#9: add a note about an uncovered case # This is the commit message astropy#10: flake # This is the commit message astropy#11: add changelog entry # This is the commit message astropy#12: add tests for log level (to test progressbar) # This is the commit message astropy#13: add back removed tests # This is the commit message astropy#14: whitespace # This is the commit message astropy#15: raise an exception if we find a cached file we can't validate
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Added a besancon reader that makes use of astropy.io.ascii, but does not require its own class (it just uses the FixedWidth reader)