Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

split physio signals into runs with a GUI#435

Open
mixue-t wants to merge 13 commits intophysiopy:masterphysiopy/phys2bids:masterfrom
mixue-t:mastermixue-t/phys2bids:masterCopy head branch name to clipboard
Open

split physio signals into runs with a GUI#435
mixue-t wants to merge 13 commits intophysiopy:masterphysiopy/phys2bids:masterfrom
mixue-t:mastermixue-t/phys2bids:masterCopy head branch name to clipboard

Conversation

@mixue-t
Copy link
Copy Markdown

@mixue-t mixue-t commented Jan 18, 2023

added functions to split physio signals into runs using peakdet GUI
peakdet is yet to be imported in slice4phys

Closes #

Proposed Changes

Change Type

  • bugfix (+0.0.1)
  • minor (+0.1.0)
  • major (+1.0.0)
  • refactoring (no version update)
  • test (no version update)
  • infrastructure (no version update)
  • documentation (no version update)
  • other

Checklist before review

  • I added everything I wanted to add to this PR.
  • [Code or tests only] I wrote/updated the necessary docstrings.
  • [Code or tests only] I ran and passed tests locally.
  • [Documentation only] I built the docs locally.
  • My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
  • My code respects the adopted style, especially linting conventions.
  • The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
  • I added or indicated the right labels.
  • I added information regarding the timeline of completion for this PR.
  • Please, comment on my PR while it's a draft and give me feedback on the development!

@smoia smoia moved this to Backlog in Software development Mar 4, 2026
@smoia smoia moved this from Backlog to Waiting for review in Software development Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: PR needs review
Status: Waiting for review
Status: PR to review

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.