Repository navigation
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Missing-value filters can return all data, and transition timestamps are incorrectly included.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds string sample-log filtering, validation, and corrected filter handling in histogram/output generation.
Changes:
- Supports case-insensitive string predicates and HDF5 string logs.
- Sorts and applies include/exclude ranges correctly.
- Adds tests, plotting support, and documentation.
File summaries
| File | Description |
|---|---|
src/test_utils.rs |
Adds mock sample-log creation. |
src/stats.rs |
Propagates log-filter errors. |
src/interface.rs |
Exposes string filtering. |
src/filters/mod.rs |
Exports predicates. |
src/filters/api.rs |
Adds string predicates and errors. |
src/data/save/sample_logs.rs |
Saves and filters string logs. |
src/data/sample_logs.rs |
Loads, filters, and sorts log ranges. |
src/data/nexus_data.rs |
Returns string logs to Python. |
src/batch_interface.rs |
Validates and applies string filters. |
MNeuEventLib/plotting.py |
Plots categorical logs. |
docs/source/how-to/filtering.ipynb |
Documents text filtering. |
Review details
- Files reviewed: 11/11 changed files
- Comments generated: 2
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
👋 Hi, @alexhroom, Conflicts have been detected against the base branch. Please rebase your branch against the base branch. This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
41cb2e9 to
f70f6ad
Compare
|
👋 Hi, @alexhroom, Conflicts have been detected against the base branch. Please rebase your branch against the base branch. This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
f70f6ad to
ba100d0
Compare
|
👋 Hi, @alexhroom, Conflicts have been detected against the base branch. Please rebase your branch against the base branch. This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
ba100d0 to
0ad6b9d
Compare
This PR adds the ability to filter sample logs which are strings. Fixes #70.
Also fixes a bug where sample log filters are filtered wrong in output files as
apply_filtersignores whether the time filters are include or exclude, and adds some validation for sample logs when the filter is created rather than when it is calculated.also now sorts sample log times. fixes #100