1. 06 Aug, 2021 3 commits
  2. 05 Aug, 2021 2 commits
  3. 04 Aug, 2021 4 commits
  4. 03 Aug, 2021 3 commits
    • Kipp Cannon's avatar
      lalburst: python 3 update · 2820fe55
      Kipp Cannon authored
      - now that python 3 is required, some clean-up can be done
      - remove "u" flags from string constants in XML I/O paths (all strings are unicode in python 3)
      - remove "from __future__ import print_function" everywhere
      - drop DictMixin compatibility code
      - adopt the "yield from ..." construct
      - update some neglected comments here and there
    • Kipp Cannon's avatar
      rate.py: various python 3 updates · 54563038
      Kipp Cannon authored
      - now that python 3 is required, some clean-up can be done
      - remove "u" flags from string constants in XML I/O paths (everything is unicode in python 3)
      - remove __ne__() method from rate.Bins (automatically implemented using __eq__() in python 3)
    • Kipp Cannon's avatar
      rate.py, iterutils.py: edit some comments · d4f2da0d
      Kipp Cannon authored
  5. 02 Aug, 2021 1 commit
  6. 01 Aug, 2021 2 commits
  7. 30 Jul, 2021 5 commits
  8. 29 Jul, 2021 4 commits
    • Adam Mercer's avatar
      Merge branch 'dont-be-quiet-for-upgrade-tests' into 'master' · c929eed6
      Adam Mercer authored
      don't use -q option for upgrade tests
      See merge request lscsoft/lalsuite!1656
    • Evan Goetz's avatar
      A big cleanup of spec_avg.c code · edbeaf33
      Evan Goetz authored
      Removed a lot of commented out code, cleaned up the code for readability, and clarified comments
      Fixed a serious bug that the SFTs were being normalized twice; this is probably not intended behaviour
      Fixed a bug (or apparent bug) in the handling of gaps between SFT files for the spectrograms
      Fixed a bug for the resolution of multiple bins averaged together for the sepctrograms
      Code cleanup fixed some weird choices like a for loop not covering the last frequency bins of the SFT when making spectrograms (I suspect poor understanding from the last person editing the code)
      Removed old Crab code that is not compiled anyway, and I suspect is no longer needed
      Fixed a memory leak on program finish
      Added test code for lalapps_spec_avg, though note that the test won't work on the previous version of the code because of several bugs that are fixed in this update. I'd like to consult with Greg Mendell before this code is approved/merged
    • Adam Mercer's avatar
      don't use -q option for upgrade tests · 0503598a
      Adam Mercer authored
      this silences the output and the list of packages that are installed,
      removed, upgraded, etc... can't be seen which makes checking that the
      upgrade is working correctly difficult.
    • Adam Mercer's avatar
      Merge branch 'remove-python2' into 'master' · 3097d4e5
      Adam Mercer authored
      Remove python2 packages
      See merge request lscsoft/lalsuite!1635
  9. 28 Jul, 2021 12 commits
  10. 27 Jul, 2021 4 commits
    • Kipp Cannon's avatar
      snglcoinc: fix some bugs · 27bb129e
      Kipp Cannon authored
      symptom:  string search fails to form triples from doubles approximately 50% of the time (50% of the triples are missing);  the inspiral search has no problem but the code is identical;  the only obvious difference between them is that the inspiral search is generally run with offset vectors containing only +ve offsets while the string search is run with +ve and -ve offsets.
      I'm not exactly sure which of these issues was the culprit, but the following issues were found and fixed and the problem in the string search was fixed:
      - when constructing doubles, construct and return all doubles containing events that can pass coincidence with an event < the boundary time, not just doubles with one of their own events < boundary time (I believe this was the only real bug in the code)
      - that required adding a filter to the final output stage to remove coincs that do not contain at least one event < the requested boundary time.
      - when assigning offset vectors to graph nodes, only normalize the entire offset vector do not normalize the components assigned to component nodes
      - teach the singlesqueue's what their time slide offset is so they can report a t_coinc_complete that accounts for the offset (this wasn't a bug, it just makes the logic easier to understand)
      - provide each singlesqueue with a coinc window unique to that instrument (this doesn't fix a bug, it addresses a FIXME to reduce latency)
      - add lots of comments explaining how the algorithm works and change some variable names to help clarify what is happening at each stage
    • Kipp Cannon's avatar
      burca.py: a few tweaks · d9dd8b3a
      Kipp Cannon authored
      - pass +inf for t_complete instead of the time of the last event to guard against edge effects due to "<" vs "<=" issues in comparisons
      - apply ntuple_copmare() inside the coinc engine using the existing feature for this.
      - don't forget to pass --min-instruments to coinc engine
    • Kipp Cannon's avatar
    • Kipp Cannon's avatar
      snglscoinc: rearrange class definition order · f92247db
      Kipp Cannon authored