Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Conversation

GianlucaFicarelli
Copy link
Collaborator

If node_id in the neuron_classes configuration is set to an empty list, it's now considered as an empty selection instead of selecting all the neurons.

@GianlucaFicarelli GianlucaFicarelli self-assigned this Mar 22, 2024
If node_id in the neuron_classes configuration is set to an empty list, it's now considered as an empty selection instead of selecting all the neurons.
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 85.04%. Comparing base (240fe50) to head (672e6ca).

Files Patch % Lines
src/blueetl/repository.py 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   85.05%   85.04%   -0.01%     
==========================================
  Files          45       45              
  Lines        2509     2508       -1     
  Branches      495      495              
==========================================
- Hits         2134     2133       -1     
  Misses        309      309              
  Partials       66       66              
Flag Coverage Δ
pytest 85.04% <86.66%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@GianlucaFicarelli GianlucaFicarelli merged commit 6d643c2 into main Mar 22, 2024
@GianlucaFicarelli GianlucaFicarelli deleted the fix_empty_node_id_list branch March 22, 2024 09:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants