notebooks icon indicating copy to clipboard operation
notebooks copied to clipboard

Published 20 hours ago • verified publisher icon spacetelescope

TST: Fix DrizzlePac align_multiple_visits

Open pllim opened this issue 5 years ago • 3 comments

Currently, CI is failing because align_multiple_visits notebook uses glob to provide inputs for TweakReg. Unfortunately, glob order can be arbitrary and not well suited for the purpose of running it in a CI. The solution is to provide a deterministic list of input files via .lis file.

Related issues: #77, #78

pllim avatar Mar 20 '19 19:03 pllim

You would also need to un-exclude the notebook so CI would pick it up again. I excluded it in 18ba024 .

pllim avatar Mar 20 '19 19:03 pllim

align_mosaic might have the same problem, as it failed in an unrelated PR in #88.

https://spacetelescope.github.io/notebooks/notebooks/DrizzlePac/align_mosaics/align_mosaics.html

pllim avatar Mar 25 '19 15:03 pllim

See #89 for the fix - currently WIP but hopefully this test will solve the issue we were running into.

catherine-martlin avatar Mar 26 '19 15:03 catherine-martlin