ContextLab / ContextLab/supereeg

possible change: remove thresholding from brain objects

Open
#119 0 comments 0 reactions 0 assignees View on GitHub
discussion help wanted question
Dominant language
Python
Stars
38
Forks
19
PR merge metrics
No merged PRs in 30d

Description

Instead of having kurtosis thresholding built into brain objects, we could write a separate function that takes a brain object, filter and threshold and returns a new filtered brain object. this would make brain objects more generalizable outside of the supereeg context and simplify the brain object code. the downside is that we'd have to add an extra line of code, and make a copy of the data, which will be problematic for (very) large datasets

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue names no files, tests, or entry points. First locate the brain object implementation and its kurtosis thresholding behavior, then clarify the separate function's API, copying requirements, and performance expectations before defining tests and what completion means.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
machine-learning
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.