-
Notifications
You must be signed in to change notification settings - Fork 20
Merging Latest into Main (V2.0.2) #125
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.
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Modified feature extraction to use new FeatureExtractor interface.
Replaced list of features with FeatureExtractor. This allows the user to pass in a FeatureExtractor object with feature parameters + standardization.
Added offline regression example .md and .rst files. Also added link to this example in index.rst.
Main --> Develop
This reverts commit 353e286.
Class name was accidentally changed during a previous commit. Reverted so its name is more consistent with other classes.
Previous commit added a duplicate write_output method definition in OnlineStreamer.
Some time series models operate by predicting on a sequence of windows instead of raw EMG data. Added option to queue windows instead of just passing in a single window.
sphinx-doc parsing wasn't detecting classes that didn't have docstrings before the constructor. Moved constructor docstrings to reflect this.
Added regression details to tabs that just described classification.
Feature queue parameter was in OnlineStreamer, but not the online classifier and regressor. Added documentation to those classes and implemented parameters to create models that feature queue.
Channels could not be isolated for online data. Added functionality to only grab certain channels online.
This reverts commit ec52763.
Rename ColumnFetch to ColumnFetcher
Offline Regression Example
Default value for regex helper function often threw an error when searching for a pattern. Replaced the default value with the wildcard, so users can use this to grab the potential values without knowing them.
Since the regex helper function can take None values, users may pass in None to RegexFilter. This wouldn't work since we store metadata as an index of the values they pass in. Added a check to ensure this doesn't happen.
Default value for regex helper function often threw an error when searching for a pattern. Replaced the default value with the wildcard, so users can use this to grab the potential values without knowing them.
Since the regex helper function can take None values, users may pass in None to RegexFilter. This wouldn't work since we store metadata as an index of the values they pass in. Added a check to ensure this doesn't happen.
An error was thrown if our method of calculating steady state frames failed (like in cases where there weren't any steady state frames). Added a try catch to deal with this.
* Add HyserMVC Dataset All Hyser datasets had been included except for MVC. Added MVC in case users wanted it for normalization. * Fix train/test split for HyserMVC
Many of these parameters were for testing and/or primarily for regression. Reduced dwell time and made timeout default to None to apply to most use cases. Also made num_trials a required parameter.
adding axis to the delsys streamer was done to the wrong location
Fitts Improvements
Online `visualize()` Bug Fix
* Add single_axis parameter * Remove manual y-axis limits Some labels may not be in the range (-1, 1) so manually setting labels wasn't best. * Add docstring for visualize method
* Add mindrove to requirements * Add MindroveStreamer * Rename mindrove.py to _mindrove.py * Pass correct append method to smm * Add comment * Add comment * Add mindrove streamer function * Add cleanup to MindroveStreamer * Improve variable name * Cleanup Mindrove streamer * Add proper cleanup for MyoStreamer * Handle proper cleanup for EMaGer streamer * Add MindRove to documentation
Added parameters ble_power and memory_mode to function configure and in the __init__ as this is an important parameter that users should be able to set when using the devices. Co-authored-by: ulysseTM <156504252+ulysseTM@users.noreply.github.com>
* added support of OTB muovi+ to the documentation * added working streamer and shared memory for OTB Muovi+
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.