Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dev #25

Merged
merged 77 commits into from
Sep 26, 2024
Merged

Dev #25

merged 77 commits into from
Sep 26, 2024

Conversation

cboulay
Copy link
Contributor

@cboulay cboulay commented Sep 24, 2024

Many changes.

  • improve docs
  • More passthrough support
  • bug fixes
  • improve unit tests
  • deduplicate code (GenAxisArray)
  • add filterbank
  • add cwt
  • switch from poetry to uv
  • add linting
  • Setup CI / CD

A note about CD: the version number is pulled from the latest git tag. When publishing to pypi using GitHub release on a 'fresh' tag, the version will come from that tag number. If installing locally (e.g., uv sync) then the version will be the latest tag + a hash if there have been commits since that tag.

@cboulay
Fixed bug in aggregate test that kept testing the same func over and …
a2fcb43 
…over
@cboulay
Add sigproc to docs.
dccb265
@cboulay
Give CAR passthrough mode.
bb92dbf
@cboulay
scaler - Support size-0 arrays. Fixes #114
5feaef3
@cboulay
Fix floating point error when calculating number of samples in a wind…
f900c7e 
…ow in sampler.
@cboulay
windowing generator now yields AxisArray. Can be composed with othe…
c1624da 
…r generators.
@cboulay
affinetransform use typing namespace.
bb29268
@cboulay
windowing - update newaxis argument docstring
2904390
@cboulay
Upgrade downsample
abf8e34 
* faster unit tests on core processing
* accommodate size-0 arrays
* slight optim. using template struct.
@cboulay
Add GH workflows.
efaa5ac
@cboulay
Fixup publish workflow
028e6ad
@cboulay
poetry.lock
99c25d7
@cboulay
ezmsg dep from github dev branch
3c023c1
@cboulay
poetry.lock
ba3ec89
@cboulay
Support a "labels" field in AxisInfo if present. Units that transform…
10bed6f 
… array shape along dimension with labels must also modify labels field.
@cboulay
Add unit tests to assert non-mutating
da142ad
@cboulay
Do a pass on gen functions to make sure they are non-mutating.
721add8
@cboulay
Update deps
36acf75
@cboulay
Use GenAxisArray as base for most sigproc units, and set zero_copy=True.
ea1e5ed
@cboulay
ranged_aggregate argmin and argmax
0643807
@cboulay
Add new units that perform simple math transformations on data only.
95dc54c
@cboulay
Add unit tests for simple math functions.
8789735
@cboulay
Fixup imports in math submodule.
5dfba54
@cboulay
Fix missing import
737d925
@cboulay
Drop "win" axis from Spectrum when returning messages without win dim.
46ba187
@cboulay
Bugfix spectrum.
ebb3049
@cboulay
Add simple integration tests for EEGSynth and Spectrum systems.
4cd1b72
@cboulay
Fixes #12 by simplifying Spectrum unit.
2cf1a13
@cboulay
Add passthrough mode for affinetransform
403baa8
@cboulay
Remove deprecated import.
e430da2
@cboulay
annot->assign
3d92354
@cboulay
test_wavelets - scratch shouldn't be run during tests.
13ce9af
@cboulay
message .axes uses frozendict in tests
cb76e0e
@cboulay
poetry.lock
d5c457d
@cboulay
Add Abs node.
c6f9427
@cboulay
Update deps, especially ezmsg
7b08549
@cboulay
version bump
d19ca23
@cboulay
Fixup tests importing utils
4354d84
@cboulay
Refactor from poetry to uv
8c702e7
@cboulay
Comment out linter for now
fc12136
@cboulay
Pytest ignore warnings
ce7edc0
@cboulay
affine test allclose instead of array_equal
7e6afbf
@cboulay
test_aclock_agen give a little more buffer for clock wonkiness
3f58bf9
@cboulay
Standardize on msg_in and msg_out instead of axis_arr_in axis_arr_out…
7427f11 
…. Saves a few characters and is more future-proof if class name changes.
@cboulay
Redo all state variables and input change checks.
811bd02
@cboulay
Fix linting in src
fe882e9
@cboulay
Fix linting in tests
dd0ad64
@cboulay
Enable CI linting and checkes on PRs against dev.
33efee9
@cboulay
Normalize imports
7f8c51a 
* group stdlib, 3rd-part, in-package
* relative for in-package
* import typing instead of from typing import
@cboulay
Dummy print in test.
4f4abd4
@cboulay
Add ruff pre-commit
9f60d9a
@cboulay
Update README
1a1c7b8
@cboulay cboulay marked this pull request as ready for review September 24, 2024 04:28
@pperanich
Copy link

It all looks good to me. I did notice that though ruff format is in the pre-commit hook, there are still a number of files that are reformatted when that hook is ran. May want to manually run the format on all files prior to merging this PR @cboulay

@griffinmilsap
Copy link
Contributor

I support merge of this PR :)

@cboulay cboulay merged commit 604b693 into main Sep 26, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pperanich pperanich pperanich approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cboulay @pperanich @griffinmilsap