IGWN Alert Client merge requestshttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests2024-02-23T21:57:09Zhttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/29igwn-alert-0.6.12024-02-23T21:57:09ZAlexander Paceigwn-alert-0.6.1- [X] test server inputs for possible concatenation errors
- [X] documentation fix- [X] test server inputs for possible concatenation errors
- [X] documentation fixhttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/28Update link to event and superevent IGWN-Alert descriptions2024-02-21T15:01:04ZCody MessickUpdate link to event and superevent IGWN-Alert descriptionshttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/27coverage badge fix2024-02-07T17:21:53ZAlexander Pacecoverage badge fixFor some reason the coverage badge broke. Adding a `coverage: ` regex fixes it.For some reason the coverage badge broke. Adding a `coverage: ` regex fixes it.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/26igwn-alert-0.6.02024-02-14T01:36:25ZAlexander Paceigwn-alert-0.6.0- Catch `KafkaException`s
* Maintain client connection for non-fatal exceptions
* Optionally restart client for fatal exceptions
- Improve logging- Catch `KafkaException`s
* Maintain client connection for non-fatal exceptions
* Optionally restart client for fatal exceptions
- Improve logginghttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/25igwn-alert-0.5.02024-01-24T18:54:28ZAlexander Paceigwn-alert-0.5.0changes:
1) expose `.flush()` from `hop-client`
2) testing changes
3) documentation fixeschanges:
1) expose `.flush()` from `hop-client`
2) testing changes
3) documentation fixeshttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/24igwn-alert-0.4.02023-10-27T18:28:42ZAlexander Paceigwn-alert-0.4.0Changes:
- Auth fixes (https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/18)
- Documentation fixes
- Unit test changesChanges:
- Auth fixes (https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/18)
- Documentation fixes
- Unit test changeshttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/23Add CI test job for Python 3.122023-10-27T15:19:44ZDuncan Macleodduncan.macleod@ligo.orgAdd CI test job for Python 3.12This MR adds a test job for Python 3.12 to the CI pipeline.This MR adds a test job for Python 3.12 to the CI pipeline.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/22Don't allow important CI jobs to fail2023-10-26T15:48:13ZDuncan Macleodduncan.macleod@ligo.orgDon't allow important CI jobs to failThis MR removes all `allow_failure` declarations from the CI pipeline, which hides important issues.This MR removes all `allow_failure` declarations from the CI pipeline, which hides important issues.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/21Test multiple versions of python2023-10-24T14:00:34ZDuncan Macleodduncan.macleod@ligo.orgTest multiple versions of pythonThis MR patches the CI configuration to test multiple versions of Python, which exposes a failure on Python 3.11.This MR patches the CI configuration to test multiple versions of Python, which exposes a failure on Python 3.11.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/20Updating old URLs in package metadata2023-09-12T00:46:25ZDaniel WysockiUpdating old URLs in package metadataReplaces dead `docs.ligo.org` links with RTD.
Also replaces GitLab links from `lscsoft/igwn-alert` (which still redirect properly) to `computing/igwn-alert/client`.Replaces dead `docs.ligo.org` links with RTD.
Also replaces GitLab links from `lscsoft/igwn-alert` (which still redirect properly) to `computing/igwn-alert/client`.Daniel WysockiDaniel Wysockihttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/19Fixing indentation in .readthedocs.yaml2023-09-11T18:18:41ZDaniel WysockiFixing indentation in .readthedocs.yamlLooks like when editing the [example `.readthedocs.yaml`](https://docs.readthedocs.io/en/stable/config-file/v2.html), the `python` block at the end was un-commented, but the extra space after the `#` was left in, breaking the indentation.Looks like when editing the [example `.readthedocs.yaml`](https://docs.readthedocs.io/en/stable/config-file/v2.html), the `python` block at the end was un-commented, but the extra space after the `#` was left in, breaking the indentation.Daniel WysockiDaniel Wysockihttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/18Address auth-related issues in API/CLI2023-10-27T18:28:42ZPatrick GodwinAddress auth-related issues in API/CLIAPI:
* Fix issue where `auth` kwarg is ignored in certain cases.
CLI:
* Modify flag for unauthenticated access from `--auth {true,false}` to `--no-auth` for clarity.
* The `--netrc` and `--username` arguments were specified but not co...API:
* Fix issue where `auth` kwarg is ignored in certain cases.
CLI:
* Modify flag for unauthenticated access from `--auth {true,false}` to `--no-auth` for clarity.
* The `--netrc` and `--username` arguments were specified but not connected to anything, so now they work a bit more as you'd expect.
Closes #13.
Closes #26.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/17Fix typo in client docstring2023-05-06T20:28:46ZCody MessickFix typo in client docstringhttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/16igwn-alert-0.3.02023-04-07T19:25:12ZAlexander Paceigwn-alert-0.3.0* backend package update
* timeout fix for improved memory usage
* consumer group support* backend package update
* timeout fix for improved memory usage
* consumer group supporthttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/15Add consumer group support for client2023-01-27T21:08:51ZPatrick GodwinAdd consumer group support for clientThis has been implemented as a `consumer_group` keyword argument for the client, allowing the client to start back up where they left off in between sessions to avoid missing messages. This can also be used to parallelize the processing ...This has been implemented as a `consumer_group` keyword argument for the client, allowing the client to start back up where they left off in between sessions to avoid missing messages. This can also be used to parallelize the processing of messages across multiple clients, in which messages are load-balanced across all consumers assigned to the same consumer group ID.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/14Increase batch timeout default for client2023-01-27T21:03:24ZPatrick GodwinIncrease batch timeout default for clientChanging this default to 0.2s tends to completely mitigate issues seen in !18. My guess is that selecting a timeout close to the round-trip time is causing the client to needlessly reconnect often. Why this causes a memory increase is be...Changing this default to 0.2s tends to completely mitigate issues seen in !18. My guess is that selecting a timeout close to the round-trip time is causing the client to needlessly reconnect often. Why this causes a memory increase is beyond me, but regardless, the new default should be a reasonable tradeoff.
Also document the `batch_size` and `batch_timeout` options in the client.
Closes #18.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/13Unpin hop-client2022-09-22T19:13:04ZLeo P. SingerUnpin hop-clientSee emfollow/gwcelery!920.See emfollow/gwcelery!920.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/12Add a running attribute to listener2022-09-21T16:59:31ZDeep Chatterjeedeep.chatterjee@ligo.orgAdd a running attribute to listenerAdding a `running` attribute to the client will help in shutting down the listener if running in a different thread.Adding a `running` attribute to the client will help in shutting down the listener if running in a different thread.https://git.ligo.org/computing/igwn-alert/client/-/merge_requests/11fix for hop-client upgrade2022-08-02T02:40:05ZAlexander Pacefix for hop-client upgradeAlexander PaceAlexander Pacehttps://git.ligo.org/computing/igwn-alert/client/-/merge_requests/10igwn-alert-0.2.02022-07-15T18:56:13ZAlexander Paceigwn-alert-0.2.0mostly a backend package change, but a few typos fixedmostly a backend package change, but a few typos fixedAlexander PaceAlexander Pace