handle HasMassGap

assigned to @deep.chatterjee and @sushant.sharma-chaudhary

mentioned in merge request userguide!141 (merged)

approved this merge request

added 2 commits

• c354f34d - 1 commit from branch emfollow:main
• d536cd12 - handle HasMassGap

Compare with previous version

added 1 commit

• dfc52787 - update lock file

Compare with previous version

@sushant.sharma-chaudhary would you please take over this MR since we have the go ahead from the reviewers?

added 35 commits

• dfc52787...132df0d7 - 32 commits from branch emfollow:main
• 850e9677 - handle HasMassGap
• f1a96385 - update lock file
• 15445303 - update lock file for v1.1.0.dev1

Compare with previous version

marked this merge request as ready

changed the description

@sarah.antier @michael-coughlin The patch is ready to be deployed in gwcelery. Please take a look.

@cody.messick Hi Cody, The patch to include mass-gap is ready. Could you review it?

requested review from @cody.messick

requested review from @sarah.antier, @geoffrey.mo, and @michael-coughlin

I have deployed this on emfollow-test for testing. Here is an example annotated superevent: https://gracedb-test.ligo.org/superevents/MS221213w/view/

@cody.messick can check all versions of dependencies in poetry.lock as astropy, matplotlib etc. I am not expert to provide an opinion about it. in orchestrator @deep.chatterjee @sushant.sharma-chaudhary can you confirm you need chi1 and chi2 compared to previous version pls ? in "ligo.em-bright" = "==1.1.0.dev1" # testing the mass-gap --> #add HasMassgap ? @deep.chatterjee ? The rest is valid @michael-coughlin

added 3 commits

• 15445303...fafe5905 - 2 commits from branch emfollow:main
• ec297a6d - handle HasMassGap

Compare with previous version

@cody.messick @geoffrey.mo The patch to include mass-gap is ready. Can someone please review it?

resolved all threads

added 1 commit

• df959e53 - handle HasMassGap

Compare with previous version