Connectivity guide update #53
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## update-0.1.1 #53 +/- ##
================================================
- Coverage 41.98% 40.30% -1.69%
================================================
Files 24 26 +2
Lines 724 794 +70
Branches 84 87 +3
================================================
+ Hits 304 320 +16
- Misses 418 472 +54
Partials 2 2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
skim0119
changed the title
skim0119
requested changes
Jun 17, 2022
There was a problem hiding this comment.
- I updated how to setup the workspace here. Please have it setup. You'll also need extra installation for documentation.
- Take a look at this one as well, especially second point in
key things to remember. - Add some descriptions and references inside the documentation as well. (example)
- Rename the file:
granger_causality_psd_cpsd_coherence. (no capital) - Plotting function can be moved to
miv/visualization/connectivity.py(or you name it). Take a look at other files in the same folder for examples.
|
Added functions for causality, causality plots and spectral plots. Updated the init files in statistics and visualization. The documentation is added to docs/guide |
…into doc_connectivity
skim0119
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Code for Granger Causality (Pairwise), Power and cross power spectral density, and welch coherence are added