[WIP] Updating the input for tweakreg #89
Conversation
|
Might need a squash so that the uncleared notebook doesn't make it into |
catherine-martlin
force-pushed
the
fix_align_multiple_visits
branch
from
698c641 to
ff61a96
Compare
|
@catherine-martlin , try rebase to use the longer timeout limit that Josh implemented earlier. |
notebooks/DrizzlePac/align_multiple_visits/align_multiple_visits.ipynb
Outdated
Show resolved
Hide resolved
- Added code to list available tables - Decreased search radius for KQ UMa to 1 arcsec (was 1 arcmin)
Co-Authored-By: theresadower <dower@stsci.edu>
Had originally avoided astroquery.mast dependencies, but the notebook is already dependent on astroquery.utils, and this change removed some other dependencies the inline function had used.
From Greg Snyder, MASTVO-293 in JIRA
PanSTARRS 1 DR2 TAP notebook
…ine-martlin/notebooks into fix_align_multiple_visits
* Copy from DA5 notebooks * Clean up NIRCam notebook, add requirements * Address review comment * Use logo added in spacetelescope#86
|
Something isn't right with the commit history... Try this: |
|
@pllim - Thank you! Definitely still figuring out how to properly rebase all the different ways when working on my fork and branch. Do you think that cleared it up appropriately? Edit: The checks just finished though, and it looks like one of the TESS notebooks failed this time - could I have missed some needed updates when I just rebased with upstream/master? |
|
@catherine-martlin , the history still doesn't look quite right. Ideally, you should only see your own relevant commits as part of this PR in the commit history. |
Changed list input for tweakreg to be hardcoded list instead of using glob as different systems can end up with files in different orders.