This repository has been archived by the owner on Oct 30, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Change main structure of the repo
Extract file names
This reverts commit 5e61c49.
Improve analysis
Merge dev into main
* Add code to pool directions * Add notes * Cleanup * Fix broken tests * Recalculate outputs * Update manifest and pyproject * Fix package name * Update package links * Update dependencies * Add more descriptions * Edit pyproject file * Expand readme with more details on the analysis * Add docstring * Test if we can see the schema in the readme file * Update docstring * Insert correct image * Linting * Apply suggestions from code review Co-authored-by: Niko Sirmpilatze <[email protected]> * Change prompt message * Adjust input params of method get_median_subtracted_response_and_params * Renaming and other small suggested fixes * Small fixes * Move out the method to make gaussian matrices * Clean up --------- Co-authored-by: Niko Sirmpilatze <[email protected]>
* Add code to pool directions * Add notes * Cleanup * Fix broken tests * Recalculate outputs * Update manifest and pyproject * Fix package name * Update package links * Update dependencies * Add more descriptions * Edit pyproject file * Expand readme with more details on the analysis * Add docstring * Test if we can see the schema in the readme file * Add handling of multiple days * Improve performance to deal with very large files, from O(n ^2) to O(1) * Delete commented code * Handle fit fails * Save the data in a given folder * Skip ROIs that do not fit * Add missing fit handling in sftf analysis * Regenerate mock data according to new "day" values * Update docstring * Remove big chunk of duplicated code * Test image in readme file * Remove image * Apply suggestions from code review Co-authored-by: Niko Sirmpilatze <[email protected]> * Change prompt message * Adjust input params of method get_median_subtracted_response_and_params * Renaming and other small suggested fixes * Renaming 2 * Small fixes * Move out the method to make gaussian matrices * Clean up * Add tests for multiple days and add more structure and comments regarding creation of mock data * Update readme * Small updates in the docs * Simplify fitting methods according to discussion * Improve description on dataset creation optimization * Remove unused config * Rearrange methods in generate mock data * Renaming ad adding new fixture * Add missing n_roi fixture calls * Renaming and folder reorganization * Remove try-catch construct * Linting * Add missing adjustments * Add readme file with a description of the regression tests * Add more descriptions / docs * Revert readme change * Fix usage of conftest * Linting --------- Co-authored-by: Niko Sirmpilatze <[email protected]>
Co-authored-by: Niko Sirmpilatze <[email protected]>
Co-authored-by: Niko Sirmpilatze <[email protected]> Co-authored-by: Joe Ziminski <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Change commit hash * Revert "Change commit hash" This reverts commit b1aa762. * Add changes introduced by cruft * Getting rid of the docs for now * Remove sphinx docs action for now * Include coverage for codecov action * Test fixing check-manifest error
Co-authored-by: Niko Sirmpilatze <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
for more information, see https://pre-commit.ci
…nit/load-suite2p into load-raw-suite2p-data
Codecov Report
@@ Coverage Diff @@
## main #81 +/- ##
==========================================
- Coverage 77.49% 74.04% -3.45%
==========================================
Files 14 16 +2
Lines 773 809 +36
==========================================
Hits 599 599
- Misses 174 210 +36
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.