OK, So the problem is in test_dcrAssembleCoadd. The config is being passed as a positional arg rather than a keyword arg:
self.config = DcrAssembleCoaddConfig()
|
self.task = DcrAssembleCoaddTask(self.config)
|
This Gen3 PipelineTask ecosystem assumes Task's have no positional arguments. Your unit test isn't standard usage, but you're surely not the only one who has passed a config in like this. I bet there are notebooks out there that do it too, so I consider this (PipelineTask conversion) an API change. I'm deciding on how to best handle this. Catch any args not in the kwargs, put them in the kwargs, and raise a Deprecation warning?
I'm really grateful for your unittest, because someone someday would be surprised about why their config wasn't being respected.
Hi Ian, You're welcome to get started reviewing this. The unit test for DCR is failing, but I THINK I just did something stupid with the _init_()'s with the multiple inheritance. I don't have time to look into that until later today.
Branches on pipe_base, pipe_tasks.
I've been informed that pipe_supertask is obsolete, so don't look in there