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

Timing lsl #337

Merged
merged 52 commits into from
Jun 17, 2024
Merged

Timing lsl #337

merged 52 commits into from
Jun 17, 2024

Conversation

SamedVossberg
Copy link
Contributor

  • changed times and logging
  • excluding parallel processing

toni-neurosc and others added 30 commits May 16, 2024 00:29
@toni-neurosc
Create Settings class with Pydantic validation
5a8f923
@toni-neurosc
Fixed a lot of bugs
b303016
@toni-neurosc
add pydantic dependency
81bddba
@toni-neurosc
Remove StrEnum
6c169b3
@toni-neurosc
Partially fixed tests
26529f9
@toni-neurosc
Refactor oscillatory
02cfb45
@toni-neurosc
Fix oscillatory bug
cb1ce78
@toni-neurosc
vectorize_bispectra
45d39ae
@toni-neurosc
More validation and settings refcatoring
f9dd6a8
@toni-neurosc
minor changes
f53657b
@toni-neurosc
Merge branch 'main' into settings_class
2bd0ac5
@toni-neurosc
Settings class validation finished, all tests working
7d2bbaf
@toni-neurosc
Ignore wrong order modules in __init__.py
d198cb2
@toni-neurosc
Fix tests, added retrocompability functions
e14e53b
@toni-neurosc
Add Pydantic dependency
09b7ae5
@toni-neurosc
Autoformat
fa456fa
@toni-neurosc
Vectorize bursts calculation
1c16150
@toni-neurosc
Fix bug
9c9ae2c
@toni-neurosc
autoformat
c615e83
@toni-neurosc
typing change
36c759d
@toni-neurosc
fix enable_all_features
9f017c6
@toni-neurosc
vectorize bursts calculation, fix settings reset bug
70bcca5
@timonmerk
review pydantic PR
55e07d0
@timonmerk
Merge branch 'settings_class' of https://github.com/neuromodulation/p…
c437a87 
…y_neuromodulation into settings_class
@toni-neurosc
fix bursts merge, fix osc tests
1e889e5
@toni-neurosc
Remove getattr/setattr where possible
2108bff
@toni-neurosc
fix bug in sharpwaves
74480b2
@toni-neurosc
Add some comments, remove seglength
3f13a4f
@toni-neurosc
Replace nm_settings.json for a .yaml
ebbd217
@toni-neurosc
Refactor normalizer classes
6777221
toni-neurosc and others added 21 commits June 13, 2024 12:34
@toni-neurosc
add more comments about burst vectorization
477863d
@SamedVossberg
fix timing and logging in lsl stream
c163eb2
@toni-neurosc
Sharpwave refactor and bugfixing
4b1e243
@toni-neurosc
Speed-up sharpwaves computation
faec1f4
@toni-neurosc
fix sharpeness bug
66bf3b2
@toni-neurosc
fix nolds default settings fband name
4e1c77d
@toni-neurosc
remove prints
1da825a
@toni-neurosc
Make MNE-connectivity non-verbose
a2da5cb
@toni-neurosc
Refactor nm_normalization + Handle divide by zero in feature normaliz…
a500e01 
…ation
@toni-neurosc
Merge branch 'main' into settings_class
c003339
@toni-neurosc
Remove a file
b593938
@toni-neurosc
Fix Numpy private function calls for Numpy 2.0 update
3ffe05b
@timonmerk
add new dependencies for numpy and scipy to be compatible with numpy …
f0a10bd 
…_core methods
@timonmerk
add function to read mne io read_raw data
fe65516
@timonmerk
fix doc example sharpwave
b919160
@timonmerk
fix add feature example
133372a
@SamedVossberg
changes in timing, exclude parllel processing
0c252db
@timonmerk
Update test_feature_sampling_rates.py
45ebe54 
Remove comments in tests
@timonmerk
Update test_timing.py
61d6a72
@timonmerk
Merge branch 'settings_class' into timing_lsl
064cf56
@timonmerk
fix test
2659bc1
@timonmerk
Copy link
Contributor

Great! Many thanks @SamedVossberg!

@timonmerk timonmerk merged commit 022402b into main Jun 17, 2024
6 checks passed
@SamedVossberg SamedVossberg mentioned this pull request Jun 18, 2024
Closed
@timonmerk timonmerk deleted the timing_lsl branch June 19, 2024 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@SamedVossberg @timonmerk @toni-neurosc