Separating allSky and SSM and PE source classification
Description
Separating allSky and SSM and PE source classification
- Based on search type or PE updatem, define source classes in each of the cases.
- This is a first commit that roughly defines the properties for SSM alerts.
Checklist
Thanks for your contribution! Please complete and check off the following quality control tasks.
Changes:
-
I created this merge request from my own fork and from a branch other than main or master. -
I checked that the CI pipeline completes without errors. -
I added a change log entry (or this is a minor change that does not require a change log entry).
Whitespace:
-
No line that I added ends with trailing whitespace. -
Every file that I modified ends with one blank line.
Grammar and spelling:
-
I proofread my contribution and made sure that it is free from spelling and grammatical errors. -
I used American English rather than British English spelling. -
I used the Oxford (serial) comma in lists of three or more items. -
I minimized use of the passive voice.
Figures and code:
-
I contributed plot directive code to generate figures (or I did not contribute any figures). -
I contributed Python code that conforms to the PEP 8 style guide (or I did not contribute Python code).
Merge request reports
Activity
assigned to @bhooshan.gadre
@deep.chatterjee @sushant.sharma-chaudhary This is just a first try. I tried separating by the search type or PE.
Also, I need to define
SSM
as a term somewhere, I guess.added 1 commit
- 284d6f56 - Adding source property definistions from online PE
mentioned in merge request !242 (closed)
Now the MR contents also include relevant parts from !242 (closed) . @divya.singh @ryan.magee Please have a look.
added 1 commit
- 9ad70610 - AllSky supercedes SSM for superevent condition added
- Resolved by Bhooshan Uday Gadre
@bhooshan.gadre looks mostly good. There are a few typos and/or minor changes I don't think you want to pull in:
1: depeding -> depending
2: added an extra
-
(I don't know if that matters)3: I don't think the
i
should be in the middle of that line.4: that the either -> that either
5: this is definition -> this definition
6: in reference -> in the reference
7: from higher frequency -> from higher frequencies OR from a higher frequency
8: GstLAL, MBTA -> GstLAL and MBTA
Thank you for carrying this forward, Bhooshan. Can I close !242 (closed) now?
In Data Analysis/ Inference sections, lets make a tabular layout connecting search and source properties and its differences.
Edited by Sushant Sharma-Chaudharyadded 1 commit
- ab243a4d - Adding SSM and updating HasNS dependence on search