Skip to content
Snippets Groups Projects

Add VTinjection search

All threads resolved!

Ingest search on the Live injection on O4 data described in Reed Essick’s repo (https://git.ligo.org/reed.essick/rpo4-injections/-/tree/main/)

https://git.ligo.org/emfollow/gwcelery/-/issues/759

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • You should add the search to the docs here.

  • If I'm understanding things correctly, this line should be changed to

    and not event.search.name in {"EarlyWarning", "VTinjection"}
  • Roberto DePietri resolved all threads

    resolved all threads

  • added 1 commit

    • 63ef107f - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Roberto DePietri added 2 commits

    added 2 commits

    • 0d2ba73d - Change name of the migratio
    • aff00ab7 - Daniel sugestion + capitalization change

    Compare with previous version

  • added 1 commit

    • 80ec4f1f - Add VTInjection search with Daniel sugestion and capitalization change.

    Compare with previous version

  • Alexander Pace
  • added 1 commit

    • f32d7743 - Add VTInjection search with Daniel sugestion and capitalization change.

    Compare with previous version

  • @roberto.depietri are there any other additions to this one? And is it consistent with changes in GWCelery?

  • Roberto DePietri resolved all threads

    resolved all threads

  • The change in gwcelery was just merged a few minutes ago (emfollow/gwcelery!1390 (merged)). It use VT_SEARCH_NAME = 'VTInjection'. We're ready to go from our side and change are consisten.

  • merged

  • Alexander Pace mentioned in merge request !201 (merged)

    mentioned in merge request !201 (merged)

  • Please register or sign in to reply
    Loading