Skip to content

Conversation

@ctrltz
Copy link
Owner

@ctrltz ctrltz commented Jun 27, 2025

This PR:

  • introduces control over amplitude envelope for Hilbert-based coupling functions (for now, using either the envelope of input waveform or the envelope of a randomly generated waveform)
  • fixes tests for the von Mises coupling function - adds a fixed random state for stability, set both lower and upper bound for the expected coupling value. Close Tests for the von Mises coupling are unstable #32

And fixed the bug, the results of select_kappa example seem fine again:
kappa-to-plv_length60sec

@ctrltz ctrltz requested a review from astudenova June 27, 2025 13:18
Copy link
Collaborator

@astudenova astudenova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything seems to work well. ദ്ദി

@ctrltz ctrltz merged commit 0b5f210 into master Jul 22, 2025
6 of 7 checks passed
@ctrltz ctrltz deleted the ctrltz/83-amplitude-envelope branch July 22, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests for the von Mises coupling are unstable

3 participants