2691 Add HistogramIntensity transform

[Nic-Ma]

Fixes #2691 .

Description

This PR implemented the HistogramIntensity transform referring to other existing examples.

Status

Ready

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[Nic-Ma]

/black

[wyli]

thanks for the quick PR, could you help go through this example (https://scikit-image.org/docs/dev/auto_examples/color_exposure/plot_equalize.html) and extract equalize_hist and equalize_adapthist?

[Nic-Ma]

Hi @wyli ,

Thanks for your review.
I updated the PR to provide the equalize_hist API.
But for the equalize_adapthist, I feel the implementation is much more complicated:
https://github.com/scikit-image/scikit-image/blob/v0.18.2/skimage/exposure/_adapthist.py#L98
I am not sure whether we must support it?
If yes, implement it with numpy referring to scikit-image or just use scikit-image as optional lib?

Thanks.

[madil90]

/build

[wyli]

sure I think we can leave equalize_adapthist for another PR. would be great to prefer scikit-image's histogram (if it's installed by the user) https://github.com/scikit-image/scikit-image/blob/main/skimage/exposure/exposure.py#L181 and add a mask option, what do you think?

[Nic-Ma]

Hi @wyli ,

Sure, I updated the PR according to your comments.
Could you please help review it again?

Thanks in advance.

[Nic-Ma]

/black

[wyli]

thanks!