search

AllenInstitute / ophys_etl_pipelines

Pipelines and modules for processing optical physiology data
Other
9 stars 5 forks source link

Fix np.int deprecation #575

Closed mikejhuang closed 1 year ago

mikejhuang commented 1 year ago

Certain tests that called functions that referenced np.int was failing due to it being deprecated in numpy v1.20

codecov[bot] commented 1 year ago

Codecov Report

Merging #575 (d14560f) into main (baf85d9) will decrease coverage by 1.34%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #575      +/-   ##
==========================================
- Coverage   97.61%   96.27%   -1.34%     
==========================================
  Files         105      105              
  Lines        8249     8249              
  Branches      775      775              
==========================================
- Hits         8052     7942     -110     
- Misses        119      229     +110     
  Partials       78       78
Flag Coverage Δ
event_detection_tests ?
general_tests 96.27% <ø> (ø)

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

see 4 files with indirect coverage changes

aamster commented 1 year ago

This doesn't look to be an issue on the build. Please always create a corresponding issue when creating a PR