Forces pipeline update when LUT changes #327

Merged
merged 3 commits into from Apr 20, 2016

Conversation

Projects
None yet
2 participants
@stefanoborini
Contributor

stefanoborini commented Mar 30, 2016

Fixes #262

@prabhuramachandran

This comment has been minimized.

Show comment
Hide comment
@prabhuramachandran

prabhuramachandran Apr 1, 2016

Member

Could you please add a small test case? Thanks.

Member

prabhuramachandran commented Apr 1, 2016

Could you please add a small test case? Thanks.

@stefanoborini

This comment has been minimized.

Show comment
Hide comment
@stefanoborini

stefanoborini Apr 1, 2016

Contributor

@prabhuramachandran I could not find pre-existing tests for those classes. Am I mistaken, or if not, do you have any preference on where to create it (new module, or an existing one?)

Contributor

stefanoborini commented Apr 1, 2016

@prabhuramachandran I could not find pre-existing tests for those classes. Am I mistaken, or if not, do you have any preference on where to create it (new module, or an existing one?)

@prabhuramachandran

This comment has been minimized.

Show comment
Hide comment
Member

prabhuramachandran commented Apr 20, 2016

Thanks.

@prabhuramachandran prabhuramachandran merged commit f44f3f1 into master Apr 20, 2016

5 checks passed

codecov/project 45.37% (target 40.00%)
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@prabhuramachandran prabhuramachandran deleted the 262-imshow-ignores-colormap-kwarg-in-443 branch Apr 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment