Open troyraen opened 1 year ago
@hernandezc1 here's your next steps:
@wmwv @djperrefort @hernandezc1 @ap1605 Reminding you of, or introducing you to, this branch.
Our ELAsTiCC broker code mostly consists of a stripped-down version of our ZTF one (perhaps with some improvements like IAM permissions). It was very helpful to @hernandezc1 and I to do this during development. But perhaps now is a good time to consider merging this back into the main branch?
But perhaps now is a good time to consider merging this back into the main branch?
In general I think it is a good idea to keep the main branch as up to date and complete as possible.
@hernandezc1 Now that we have multiple surveys going in the main (develop
) branch, how about merging this branch back in there?
This issue announces the designation of
main/elasticc
as the main branch for our ELAsTiCC broker. This issue will also track the branch's lifecycle.main/elasticc
is either:develop
,develop
could becomemain/ztf
.~ (ok, probably not)Background / Motivation
The changes in PR #153 represent a logical first chunk in our development for ELAsTiCC. In addition, we have copied some of the broker's modules to the new pittgoogle-user where they will be used to run an independent pipeline in a separate GCP project, that will operate downstream from our ELAsTiCC broker. See #107.
Our ELAsTiCC broker and its development track is now considerably different from our ZTF broker. In addition, right now, our work on the ELAsTiCC broker needs to move quickly and nimbly. Thus, we are now designating
main/elasticc
as the main branch for our ELAsTiCC broker. This will allow our development for the ELAsTiCC broker to continue as an independent (sub)project -- with its own clear chain of PRs, etc. -- without cutting it off from this repo completely. We can decide whether/how to merge the two broker code bases after the ELAsTiCC broker design solidifies a bit more.Plan for the immediate future
develop
tomain/elasticc
. Right now those two branches are exactly the same, so this will not change anything about any of the code changes included in the PR. It just cuts thedevelop
branch off so elasticc won't affect it (for now).u/CH/test
->main/elasticc
). Congratulations on a successful pull request 🎉 .main/elasticc
as the main branch for the project. All work for this project should then be done in feature branches that are based onmain/elasticc
-- and thus we should open PRs for those feature branches that request to merge intomain/elasticc
.cc: @wmwv