New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-25979: IngestIndexReferenceTask throws if coord_err_unit is None #208
Conversation
800a7a0
to
154f261
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about the bug!
You mentioned that the test isn't getting run by pytest? Do we need to add something to the SConscript or somewhere?
To small suggestions to make debugging the tests easier.
tests/test_htmIndex.py
Outdated
loader = LoadIndexedReferenceObjectsTask(butler=dafPersist.Butler( | ||
self.outPath + "/output_setsVersion")) | ||
self.assertEqual(loader.dataset_config.format_version, 1) | ||
for withRaDecErr in (True, False): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please break this block out as check_version()
or some such, and then explicitly call it twice? The for
loop over two items can be a bit harder to debug.
@@ -68,6 +70,10 @@ def testIngestTwoFilesTwoCores(self): | |||
self.checkAllRowsInRefcat(loader, skyCatalog1, config) | |||
self.checkAllRowsInRefcat(loader, skyCatalog2, config) | |||
|
|||
def testIngestTwoFilesTwoCores(self): | |||
for withRaDecErr in [True, False]: | |||
self._run_testIngestTwoFilesTwoCores(withRaDecErr) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please make this two separate calls? That way, if one fails, it will be much more obvious which one.
154f261
to
9b4eaaf
Compare
9b4eaaf
to
8c1f58e
Compare
AKA “that looks more fun than proof rereading slides”.