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

Offline analysis updates for multimodal fusion. #360

Merged
merged 26 commits into from
Nov 5, 2024
Merged

Offline analysis updates for multimodal fusion. #360

merged 26 commits into from
Nov 5, 2024

Conversation

celikbasak
Copy link
Member

@celikbasak celikbasak commented Nov 4, 2024 •
edited
Loading

Overview

Multimodal fusion updates to be included in 2.0.0 release

Tickets

Multimodal
Balanced accuracy error
ET Model class
ET Model integration
ET Evidence Class

Contributions

  • Offline analysis gaze model training complete. Both EEG and gaze models are trained and saved to be used in fusion test case later. A patch will needed to complete the testing of the fusion.
  • Balanced accuracy error is solved during EEG model training.
  • Gaze models are generated complying with the SignalModel base class.
  • Gaze Evidence class added to support online use.

Test

  • Define the steps used to test your code here. Include unit and integration test steps! To be updated!

Documentation

  • Are documentation updates required? In-line, README, or documentation? Verify the updates you did here. Documentation to be updated!

Changelog

  • CHANGELOG.md updated.

celikbasak and others added 18 commits November 30, 2023 19:25
@celikbasak
Offline analysis updates, bug fixes
a7a02a8
@celikbasak
Changelog updated
2bc15db
@celikbasak
plotting updates v2
bda7072
@celikbasak
Changes to offline analysis and gaze model, GazeEvaluator added
03a07b2
@tab-cmd
add to the ignore
88951a4
@tab-cmd
Merge branch '2.0.0rc4' into matrix_copyphrase
c7d20c4
@tab-cmd
Update to Inquiry Reshaper, comply with signal base model
c07590b
@celikbasak
Updates on gaze models - will be refactored soon
99ddd36
@celikbasak
GP model added
a6d1a31
@celikbasak
changes in offline analysis
b04a523
@celikbasak
updates WIP
040f552
@celikbasak
more updates to the model
cc3933a
@celikbasak
updates on GP
14ae79c
@celikbasak
updates WIP
477d930
@celikbasak
changes Oct 28
0d8dc14
@tab-cmd
Merge remote-tracking branch 'origin/2.0.0rc4' into aaai_submission_t…
a497d96 
…bd_later
@celikbasak
Offline analysis updated to train and save each model separately. Pat…
77e3f07 
…chwork needed to complete the testing of fusion
@celikbasak
All csv files containing results are removed
b0d11b1
@celikbasak celikbasak requested review from tab-cmd and lawhead November 4, 2024 18:00
tab-cmd
tab-cmd reviewed Nov 4, 2024
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
bcipy/helpers/demo/demo_visualization.py Outdated Show resolved Hide resolved
bcipy/parameters/symbol_positions.json Outdated Show resolved Hide resolved
bcipy/signal/model/gaussian_mixture/gaussian_mixture.py Outdated Show resolved Hide resolved
bcipy/signal/model/offline_analysis.py Outdated Show resolved Hide resolved
bcipy/signal/model/plotting.py Outdated Show resolved Hide resolved
bcipy/signal/process/extract_gaze.py Outdated Show resolved Hide resolved
bcipy/task/control/handler.py Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
@tab-cmd tab-cmd merged commit a56e1d8 into 2.0.0rc4 Nov 5, 2024
0 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tab-cmd tab-cmd tab-cmd left review comments

@lawhead lawhead lawhead left review comments

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
@celikbasak @lawhead @tab-cmd