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

Wrong boys2rgb colormap #857

Open
2 tasks
zhenchen-jay opened this issue Jan 10, 2024 · 11 comments
Open
2 tasks

Wrong boys2rgb colormap #857

zhenchen-jay opened this issue Jan 10, 2024 · 11 comments
Labels
type:Bug Fix Something isn't working

Comments

@zhenchen-jay
Copy link

zhenchen-jay commented Jan 10, 2024

Description

I think there is a typo in the function boys2rgb

https://github.com/fury-gl/fury/blob/master/fury/colormap.py#L118

it is actually should be:
z4 = z2 * z2

This can be also seen in the initial matlab code: https://cagataydemiralp.io/projects/boys/line2rgb.m

Way to reproduce

  • Code example
import numpy as np
from fury import colormap

if __name__ == '__main__':
    v = np.array([[0.0183422, 0.00418003, 0.999823], [-0.0183422, -0.00418003, -0.999823]])
    print(colormap.boys2rgb(v))
  • Result
[[0.49262681 0.48131988 0.05134322]
 [0.49262681 0.48131988 0.87099833]]
@DanonymousCoder
Copy link

Hi, I would love to work on this issue

@DanonymousCoder
Copy link

@zhenchen-jay

@zhenchen-jay
Copy link
Author

cool. Which additional information I need to provide?

@DanonymousCoder
Copy link

DanonymousCoder commented Jan 13, 2024

Pls, kindly assign me this issue to work on @zhenchen-jay @skoudoro

@zhenchen-jay
Copy link
Author

I am not able to do that since I don't have the necessary permission. @DanonymousCoder

@ganimtron-10
Copy link
Contributor

Hey @zhenchen-jay ,
Thanks for pointing this out.

@DanonymousCoder if you interested, please feel free to go on and create a PR.

@DanonymousCoder
Copy link

Already created a pull request @ganimtron-10 . Pls, kindly check.

@ganimtron-10
Copy link
Contributor

Cool @skoudoro @guaje would give a look.

@skoudoro
Copy link
Contributor

will look into it this week, thanks for the review @ganimtron-10

@Preetam-Das26
Copy link

Preetam-Das26 commented Jan 27, 2024

hello is this issue active?CAn I work on it?
@skoudoro @ganimtron-10

@ganimtron-10
Copy link
Contributor

@Preetam-Das26 This is already picked up by @DanonymousCoder

@WassCodeur WassCodeur added the type:Bug Fix Something isn't working label Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Bug Fix Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants