Skip to content

BF: Fixed cielch2rgb handling of multidim arrays#6961

Merged
mdcutone merged 3 commits intopsychopy:devfrom
mdcutone:dev-bf-cielch2rgb-multidim
Nov 11, 2024
Merged

BF: Fixed cielch2rgb handling of multidim arrays#6961
mdcutone merged 3 commits intopsychopy:devfrom
mdcutone:dev-bf-cielch2rgb-multidim

Conversation

@mdcutone
Copy link
Copy Markdown
Member

@mdcutone mdcutone commented Nov 4, 2024

Fixes #6925 by using vectorized functions

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.81%. Comparing base (23e6951) to head (52bbf27).
Report is 2 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #6961      +/-   ##
==========================================
+ Coverage   50.79%   50.81%   +0.02%     
==========================================
  Files         340      340              
  Lines       62218    62220       +2     
==========================================
+ Hits        31601    31617      +16     
+ Misses      30617    30603      -14     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@mdcutone
Copy link
Copy Markdown
Member Author

mdcutone commented Nov 8, 2024

Added a test and reduced the number of operations in the conversion function

@mdcutone mdcutone merged commit 27dd513 into psychopy:dev Nov 11, 2024
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.

[Bug]: multidimensional arrays break cielch2rgb

1 participant