Skip to content

Scale resampling length#351

Open
jo-mueller wants to merge 3 commits into
campaslab:mainfrom
jo-mueller:scale-resampling-length
Open

Scale resampling length#351
jo-mueller wants to merge 3 commits into
campaslab:mainfrom
jo-mueller:scale-resampling-length

Conversation

@jo-mueller

Copy link
Copy Markdown
Member

Description

This scales the resampling_length parameter so that the trace points numbers don't escalate for high-res data.

Fixes #349

Type of Change

  • Parameter estimation change

How Has This Been Tested?

No tests necessary - doesn't change functionality, just the way parameters are estimated

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code

@jo-mueller jo-mueller added the enhancement New feature or request label Dec 21, 2023
@codecov

codecov Bot commented Dec 21, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ab04414) 85.93% compared to head (4be69b5) 85.92%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
- Coverage   85.93%   85.92%   -0.01%     
==========================================
  Files          53       53              
  Lines        9561     9559       -2     
==========================================
- Hits         8216     8214       -2     
  Misses       1345     1345              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Demo Analyze Everything Hung Up on Reconstruction

1 participant