Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New plotting functions #163

Merged
merged 24 commits into from
Aug 13, 2024
Merged

New plotting functions #163

merged 24 commits into from
Aug 13, 2024

Conversation

AurelienJaquier
Copy link
Collaborator

@AurelienJaquier AurelienJaquier commented Jul 26, 2024

  • new utility function to get extraction output pickle file paths
  • adding new plotting functions:
    • IV curve
    • FI curve
    • phase plot
    • sinespec / impedance (does not depend on responses, run its own hard-coded protocol)
  • allow new plotting functions in luigi task
  • refactored plotting function with new file: plotting_utils
    in order to shorten a little bit the plotting.py file
  • adds the current trace to efel (for impedance). it is the user responsibility to add a recording with type FixedDtRecordingStimulus or LooseDtRecordingStimulus and name {protocol_name}.iclamp.i for it to work as expected
  • added tests for most of the plotting_utils functions
  • merged test_plotting.py into new plotting_utils.py
  • linted tests/ folder (sorry, that adds a bunch of line changes)

Jaquier Aurélien Tristan added 4 commits July 25, 2024 15:44
Change-Id: I5fc800ce7b50f85f35ef232d87450432c8f80d43
Change-Id: I94067d4f6133e78a5137e9629988293f59a539df
Change-Id: I6f39334d37a6eaadfe928fa31e0c3ebf6bfedb90
Change-Id: I7abaf2ed0ffd53e2644a5c3ad52eec6b44ff6b69
@AurelienJaquier AurelienJaquier self-assigned this Jul 26, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2024

Codecov Report

Attention: Patch coverage is 41.30719% with 449 lines in your changes missing coverage. Please review.

Project coverage is 52.42%. Comparing base (8d5b0ab) to head (9771ee2).
Report is 53 commits behind head on main.

Files Patch % Lines
bluepyemodel/emodel_pipeline/plotting.py 9.74% 250 Missing ⚠️
bluepyemodel/emodel_pipeline/plotting_utils.py 50.90% 135 Missing ⚠️
bluepyemodel/evaluation/recordings.py 25.00% 15 Missing ⚠️
...model/efeatures_extraction/efeatures_extraction.py 27.77% 13 Missing ⚠️
bluepyemodel/evaluation/protocols.py 7.14% 13 Missing ⚠️
bluepyemodel/tasks/emodel_creation/optimisation.py 0.00% 9 Missing ⚠️
bluepyemodel/tools/utils.py 16.66% 5 Missing ⚠️
bluepyemodel/evaluation/evaluator.py 55.55% 4 Missing ⚠️
bluepyemodel/emodel_pipeline/emodel_pipeline.py 0.00% 2 Missing ⚠️
bluepyemodel/emodel_pipeline/emodel_settings.py 94.73% 1 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #163      +/-   ##
==========================================
- Coverage   60.07%   52.42%   -7.66%     
==========================================
  Files         109      124      +15     
  Lines        7838    10299    +2461     
==========================================
+ Hits         4709     5399     +690     
- Misses       3129     4900    +1771     

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

@AurelienJaquier AurelienJaquier marked this pull request as draft July 26, 2024 15:02
Jaquier Aurélien Tristan added 12 commits July 29, 2024 10:57
Change-Id: Ieb348192e397f5cf7072b91b3e1b1236eb9eee86
Change-Id: I4562d27d5a6a4644a4a6765af14e8ef0c7f8af23
…plotting functions

Change-Id: Id8187b27ac1a170a42c140a9345dc5cdaf3a788e
Change-Id: I32a9309341c4940e5750a80cd38776cfb6d89049
Change-Id: I5ed3d4573dc01c4c6558d272f73d9e269378706a
Change-Id: Ica49bb1d5fa2655a185a4bfdac2f004f58dac7a8
Change-Id: I463dca3e2d6c872d81eff325c17b486e6477e74e
Change-Id: I7c59e356db623239c461a8e4edca184fed0b68e7
Change-Id: I56e0c829db6edb4b231d5a5b39b920afae78377c
Change-Id: I894a0658af15731ce8be8f528d5bc8f74edd12a1
Change-Id: I75f01e37eb2d6b50309f3caa92ad3f65c964c1cd
Change-Id: I8956d44314d6750c2b1aa9332d9298f69e72faa8
@AurelienJaquier AurelienJaquier marked this pull request as ready for review August 7, 2024 14:33
Jaquier Aurélien Tristan added 4 commits August 7, 2024 17:44
Change-Id: Idb40795269a9ef6c580f2175e5354d38e61766e5
Change-Id: Icc0fae8ce116b4f61b2430519e320403583caef1
Change-Id: I03bb06fa572aa0771d4fec9d3cb63f3e26773ae1
Change-Id: I41716436518ee376bce909855a1a01bc2f30902a
Change-Id: I701eb9002dd8d759811622eb10bcc4ec91b5dbab
Jaquier Aurélien Tristan added 2 commits August 13, 2024 09:18
Change-Id: Idf667560c64e3fcba83114f9e14d5549c33fbeea
Change-Id: Iac654f81c66a7401a77fa181dd43618e499d4573
Change-Id: Idb85dcf3b0f9fc3a3777f5fd16a5d252ec70e614
Copy link
Collaborator

@ilkilic ilkilic left a comment

Choose a reason for hiding this comment

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

Great work, thank you! looks good to me

@AurelienJaquier AurelienJaquier merged commit 8387c4d into main Aug 13, 2024
6 checks passed
@AurelienJaquier AurelienJaquier deleted the iv-curve-plot branch August 13, 2024 08:20
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.

4 participants