Skip to content
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

tickets/DM-27208 #18

Merged
merged 32 commits into from Nov 3, 2020
Merged

tickets/DM-27208 #18

merged 32 commits into from Nov 3, 2020

Conversation

fred3m
Copy link

@fred3m fred3m commented Oct 28, 2020

Update to latest scarlet

pmelchior and others added 30 commits August 9, 2020 17:06
This is only part 1 of this process. Part 2 will be to make the tests
work from this new location(still in this PR),
and part 3 will be a future PR to refactor this file into multiple
files in an `scarlet/initialization` directory as follows:

* spectrum.py - contains get_best_fit_seds, get_pixel_sed, get_psf_sed from initialization.py
* morphology.py - contains `trim_morphology` and will also include some of the functionality from `init_extended_source` and `init_starlet_source`
* coadds.py - contains `build_sed_coadd` and `build_initialization_coadd`
* initialization.py - contains `init_extended_source`,  `init_starlet_source`, `init_multicomponent_source` and all of the init functions needed for the testing (`hasEdgeFlux`, `initAllSources`, `iniSource`)

But in order to not break (or conflict with) PR pmelchior#204 this change is
being held until a future PR.
This commit fixes an issue that was created when porting the tests
from scarlet_extensions to scarlet. Namely, I didn't want to have
to introduce the `git` package as a dependency so I attempted to
replicate the code on my own. That did not go so well...
So this commit reimplements the git solution that was working.
@fred3m fred3m merged commit 51dde32 into lsst-dev Nov 3, 2020
@fred3m fred3m deleted the tickets/DM-27208 branch November 3, 2020 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants