preliminary source_classification ML integration
-
source_classificationis based on ML estimates - classifiers read from emfollow/data to produce estimates per event basis
Merge request reports
Activity
added 8 commits
-
19695b4c...a78119f7 - 7 commits from branch
emfollow:master - 41e6d450 - preliminary source_classification unittests
-
19695b4c...a78119f7 - 7 commits from branch
added 4 commits
-
41e6d450...c5efa8a8 - 3 commits from branch
emfollow:master - 7397c555 - preliminary source_classification unittests
-
41e6d450...c5efa8a8 - 3 commits from branch
added Classification label
mentioned in merge request !359 (merged)
added 7 commits
-
7397c555...14db1e32 - 6 commits from branch
emfollow:master - 65db7282 - preliminary source_classification unittests
-
7397c555...14db1e32 - 6 commits from branch
@leo-singer The pickle files in the test are serving dual purpose. It firstly provides us the data for the unittests. It is also a fallback option for the EM-Bright inference, in case the connectivity fails for some reason (see: https://git.ligo.org/deep.chatterjee/gwcelery/blob/source-classification-ML/gwcelery/tasks/em_bright.py#L51-60 and https://git.ligo.org/lscsoft/p-astro/blob/master/ligo/em_bright.py#L68-76).
This might not be the best option. I will be happy hear any better suggestion.
Edited by Shaon Ghosh-
@leo-singer The pickle files in the test are serving dual purpose. It firstly provides us the data for the unittests. It is also a fallback option for the EM-Bright inference, in case the connectivity fails for some reason (see: https://git.ligo.org/deep.chatterjee/gwcelery/blob/source-classification-ML/gwcelery/tasks/em_bright.py#L51-60 and https://git.ligo.org/lscsoft/p-astro/blob/master/ligo/em_bright.py#L68-76).
This might not be the best option. I will be happy hear any better suggestion.
You could keep the files in the p-astro repository.
added 16 commits
-
65db7282...0cb75a50 - 15 commits from branch
emfollow:master - 56d8ca85 - preliminary source_classification unittests
-
65db7282...0cb75a50 - 15 commits from branch
added 4 commits
-
21fcb54d...245fa579 - 3 commits from branch
emfollow:master - b804e571 - preliminary source_classification unittests
-
21fcb54d...245fa579 - 3 commits from branch
added 1 commit
- 7878d197 - puttting back the old source classification scheme from OPA for non-gstLAL triggers
added 1 commit
- 7e5fd1fd - Added rem mass computation for the point estimate case. Fixed naming of the...
added 1 commit
- 6e269116 - don't pass coinc or psd to classifier tasks anymore
added 1 commit
- 11912a8c - added and pinned p-astro version in requirements
added 10 commits
-
11912a8c...49112f42 - 4 commits from branch
emfollow:master - 69d0dfc9 - preliminary source_classification unittests
- adb5accc - puttting back the old source classification scheme from OPA for non-gstLAL triggers
- 9f47d24a - Added rem mass computation for the point estimate case. Fixed naming of the...
- c39241ad - Fixed syntax error
- 634e7fb7 - don't pass coinc or psd to classifier tasks anymore
- f4b67ed8 - added and pinned p-astro version in requirements
Toggle commit list-
11912a8c...49112f42 - 4 commits from branch
added 11 commits
-
8fe17e70...f29f0ce2 - 4 commits from branch
emfollow:master - e12a057e - preliminary source_classification unittests
- f37c9b6e - puttting back the old source classification scheme from OPA for non-gstLAL triggers
- b8cf1774 - Added rem mass computation for the point estimate case. Fixed naming of the...
- 8a43f293 - Fixed syntax error
- 1c930cce - don't pass coinc or psd to classifier tasks anymore
- 23dcd477 - added and pinned p-astro version in requirements
- c9085a0f - increased coverage
Toggle commit list-
8fe17e70...f29f0ce2 - 4 commits from branch
