glue merge requestshttps://git.ligo.org/lscsoft/glue/-/merge_requests2024-03-01T14:02:59Zhttps://git.ligo.org/lscsoft/glue/-/merge_requests/146Build wheels for upload to PyPI2024-03-01T14:02:59ZLeo P. SingerBuild wheels for upload to PyPIFixes #41.Fixes #41.https://git.ligo.org/lscsoft/glue/-/merge_requests/113Fix lint in glue.LDBDWClient2023-11-02T15:17:32ZDuncan Macleodduncan.macleod@ligo.orgFix lint in glue.LDBDWClientThis MR fixes the following [lint](https://git.ligo.org/duncanmmacleod/glue/-/jobs/1761987) in `glue.LDBDWClient`
```text
/builds/duncanmmacleod/glue/glue/LDBDWClient.py:26:1: F401 'types' imported but unused
/builds/duncanmmacleod/glue...This MR fixes the following [lint](https://git.ligo.org/duncanmmacleod/glue/-/jobs/1761987) in `glue.LDBDWClient`
```text
/builds/duncanmmacleod/glue/glue/LDBDWClient.py:26:1: F401 'types' imported but unused
/builds/duncanmmacleod/glue/glue/LDBDWClient.py:271:5: F841 local variable 'data' is assigned to but never used
/builds/duncanmmacleod/glue/glue/LDBDWClient.py:439:1: W391 blank line at end of file
```https://git.ligo.org/lscsoft/glue/-/merge_requests/112Fix lint in glue.segmentdb subpackage2023-11-02T15:16:17ZDuncan Macleodduncan.macleod@ligo.orgFix lint in glue.segmentdb subpackageThis MR fixes lint in the `glue.segmentdb` subpackage:
- remove blank line at end of query_engine.py
- remove unused/unnecessary imports in segmentdb_utils.pyThis MR fixes lint in the `glue.segmentdb` subpackage:
- remove blank line at end of query_engine.py
- remove unused/unnecessary imports in segmentdb_utils.pyhttps://git.ligo.org/lscsoft/glue/-/merge_requests/123Don't fail flake8 job in CI2023-10-18T13:14:58ZDuncan Macleodduncan.macleod@ligo.orgDon't fail flake8 job in CIThis MR modifies the `flake8` CI job to always return a zero (`0`) exit code; we should rely on the code quality widget on the merge request UI to detect quality regressions.This MR modifies the `flake8` CI job to always return a zero (`0`) exit code; we should rely on the code quality widget on the merge request UI to detect quality regressions.Robert BruntzRobert Bruntzhttps://git.ligo.org/lscsoft/glue/-/merge_requests/120Conflict on old python3-glue-common (to get it removed first)2022-03-31T17:16:05ZSteffen GrunewaldConflict on old python3-glue-common (to get it removed first)@duncanmmacleod (!119) had the same idea, but with additional linebreaks.@duncanmmacleod (!119) had the same idea, but with additional linebreaks.https://git.ligo.org/lscsoft/glue/-/merge_requests/100Draft: Don't include glue.ligolw in Python 3.10 or later2022-01-24T12:21:31ZDuncan Macleodduncan.macleod@ligo.orgDraft: Don't include glue.ligolw in Python 3.10 or laterThis MR modifies the build configuration to exclude `glue.ligolw` from all builds on Python 3.10 or later. This works around the fact that `glue.ligolw` doesn't run on that platform and nobody is going to fix it.
Fixes #23.This MR modifies the build configuration to exclude `glue.ligolw` from all builds on Python 3.10 or later. This works around the fact that `glue.ligolw` doesn't run on that platform and nobody is going to fix it.
Fixes #23.https://git.ligo.org/lscsoft/glue/-/merge_requests/75Remove ldg_submit_dax and associated files2021-10-12T08:55:41ZDuncan Macleodduncan.macleod@ligo.orgRemove ldg_submit_dax and associated filesThis MR removes the `ldg_submit_dax` script and associated files.
@chad-hanna is listed as the author of `ldg_submit_dax`, so hopefully can verify that this isn't used any more.This MR removes the `ldg_submit_dax` script and associated files.
@chad-hanna is listed as the author of `ldg_submit_dax`, so hopefully can verify that this isn't used any more.https://git.ligo.org/lscsoft/glue/-/merge_requests/69Silence Python 3.8 PY_SSIZE_T_CLEAN warning2021-06-28T16:35:58ZLeo P. SingerSilence Python 3.8 PY_SSIZE_T_CLEAN warningIn Python 3, `PyArg_ParseTuple` and friends can return either an int
or a Py_ssize_t type for the length in an argument of the form
`"s#"`, `"y#"`, `"u#"`, etc, controlled by whether the preprocessor
macro `PY_SSIZE_T_CLEAN` is defined w...In Python 3, `PyArg_ParseTuple` and friends can return either an int
or a Py_ssize_t type for the length in an argument of the form
`"s#"`, `"y#"`, `"u#"`, etc, controlled by whether the preprocessor
macro `PY_SSIZE_T_CLEAN` is defined when the header `Python.h` is
included. A future version of Python will only support `PY_SSIZE_T`.
In Python 3.8, there was a deprecation warning added (see
https://bugs.python.org/issue36381).
Define the `PY_SSIZE_T_CLEAN` macro to suppress the hundreds or
thousands of deprecation warnings that may be printed when reading
a LIGO_LW file.Kipp CannonKipp Cannonhttps://git.ligo.org/lscsoft/glue/-/merge_requests/70Add f_lower column to sngl_inspiral table2021-06-28T16:35:51ZLeo P. SingerAdd f_lower column to sngl_inspiral tablePipelines need to report their low frequency cutoff in order to
avoid luminosity distance biases.
See https://sympa.ligo.org/wws/arc/cbc/2020-04/msg00116.html.Pipelines need to report their low frequency cutoff in order to
avoid luminosity distance biases.
See https://sympa.ligo.org/wws/arc/cbc/2020-04/msg00116.html.https://git.ligo.org/lscsoft/glue/-/merge_requests/65Rebuild of !642021-06-28T15:23:55ZDuncan Macleodduncan.macleod@ligo.orgRebuild of !64This is just a rebuild of !64 (and supersedes it), using the same commits, to account for transient CI failures.This is just a rebuild of !64 (and supersedes it), using the same commits, to account for transient CI failures.Ryan FisherRyan Fisherhttps://git.ligo.org/lscsoft/glue/-/merge_requests/68Pin CI docs build to python3.72021-06-28T08:13:42ZDuncan Macleodduncan.macleod@ligo.orgPin CI docs build to python3.7Ryan FisherRyan Fisherhttps://git.ligo.org/lscsoft/glue/-/merge_requests/28Remove cacheentry2020-06-18T04:05:21ZIan HarryRemove cacheentrySee !27 (@duncanmmacleod @kipp.cannon), here, as discussed, the CacheEntry class is completely removed from glue. As 2 other modules use CacheEntry within glue there are also changes there to point to lal.utils.CacheEntry.See !27 (@duncanmmacleod @kipp.cannon), here, as discussed, the CacheEntry class is completely removed from glue. As 2 other modules use CacheEntry within glue there are also changes there to point to lal.utils.CacheEntry.2.0.0Ryan FisherRyan Fisherhttps://git.ligo.org/lscsoft/glue/-/merge_requests/66use %{python3_pkgversion} macro2019-12-06T17:00:19ZAdam Merceruse %{python3_pkgversion} macro%{python3_version_nodots} has been causing problems with the SL7.7
rebuilds%{python3_version_nodots} has been causing problems with the SL7.7
rebuildshttps://git.ligo.org/lscsoft/glue/-/merge_requests/64Install dependency on python*-glue-common, supersedes !622019-08-16T11:33:34ZSteffen GrunewaldInstall dependency on python*-glue-common, supersedes !62This MR replaces !62. Updates were performed via Web i/f, please squash commits.This MR replaces !62. Updates were performed via Web i/f, please squash commits.https://git.ligo.org/lscsoft/glue/-/merge_requests/62Fix https://git.ligo.org/lscsoft/glue/issues/162019-07-12T08:03:49ZSteffen GrunewaldFix https://git.ligo.org/lscsoft/glue/issues/16Please merge this upstream, to fix https://git.ligo.org/lscsoft/glue/issues/16Please merge this upstream, to fix https://git.ligo.org/lscsoft/glue/issues/16https://git.ligo.org/lscsoft/glue/-/merge_requests/37Add support for numerical burst injections in SimBurstTables2019-07-10T20:26:08ZDaniel Williamsdaniel.williams@ligo.orgAdd support for numerical burst injections in SimBurstTablesThe existing implementation of the SimBurstTable code does not allow for the injection of numerical relativity waveforms. This is a desirable feature for e.g. supernova and ADI waveforms. This branch implements such support in a way anal...The existing implementation of the SimBurstTable code does not allow for the injection of numerical relativity waveforms. This is a desirable feature for e.g. supernova and ADI waveforms. This branch implements such support in a way analogous to in a SimInspiralTable, and allows xml files with these columns to be read.
I'm very open to suggestions for better ways to handle this; this seemed like a good, quick solution a couple of years ago when I was preparing the O1 MDC for the burst group, but I'm prepared to make changes to minke (the burst MDC package) to allow for an alternative implementation if that seems more sensible.
Making this change to glue doesn't really make sense until support is added to LALSuite either.Kipp CannonKipp Cannonhttps://git.ligo.org/lscsoft/glue/-/merge_requests/58Fix file descriptor leaks in glue.ligolw.utils2019-02-06T14:37:16ZLeo P. SingerFix file descriptor leaks in glue.ligolw.utilsMake sure that `glue.ligolw.utils.load_{filename,url}` close any
file descriptors that they open. This will improve the stability
of long-running processes that read or write many LIGO-LW files.
The indication that they were leaking...Make sure that `glue.ligolw.utils.load_{filename,url}` close any
file descriptors that they open. This will improve the stability
of long-running processes that read or write many LIGO-LW files.
The indication that they were leaking file descriptors came from
pytest-openfiles warnings in the ligo.skymap unit tests.
CC @kipp.cannon, @ryan.fisher.https://git.ligo.org/lscsoft/glue/-/merge_requests/36Rename python-glue-ligolw-tools to glue-ligolw-tools2018-12-02T04:04:41ZLeo P. SingerRename python-glue-ligolw-tools to glue-ligolw-toolsSince these are command line tools, not modules, by convention
the package name should not start with `python-`.Since these are command line tools, not modules, by convention
the package name should not start with `python-`.https://git.ligo.org/lscsoft/glue/-/merge_requests/48Glue 1 59 2 Branch Merge2018-07-24T15:47:10ZRyan FisherGlue 1 59 2 Branch MergeDuncan Macleodduncan.macleod@ligo.orgDuncan Macleodduncan.macleod@ligo.orghttps://git.ligo.org/lscsoft/glue/-/merge_requests/38Fix base row type picklability2018-03-13T02:02:26ZLeo P. SingerFix base row type picklabilityThis should #8. The problem was that the `__setstate__` method assumed that derived classes have `__init__` methods with the same signature (a dictionary of column values).
(Also, it seems to me a little weird to directly call `__init__...This should #8. The problem was that the `__setstate__` method assumed that derived classes have `__init__` methods with the same signature (a dictionary of column values).
(Also, it seems to me a little weird to directly call `__init__` from other functions; it seems less weird to directly call `__setstate__`. That's just my opinion.)