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

Widgets bug fix and Active Template Model widget #494

Merged
merged 7 commits into from Oct 27, 2014

Conversation

Projects
None yet
3 participants
@nontas
Member

nontas commented Oct 26, 2014

This PR:

  1. Fixes the bug reported in #493 . Specifically the visualize_images widget was crashing if the first image of the provided list didn't have landmarks.
  2. Adds an extra RGB viewing option in the channel options (as proposed in #493 ). The checkbox only appears if the current image has 3 channels.
  3. Adds visualize_atm widget that follows the structure of visualize_aam.

@jabooth jabooth added the in progress label Oct 26, 2014

@nontas nontas referenced this pull request Oct 26, 2014

Closed

Widgets bugs #493

@patricksnape

This comment has been minimized.

Contributor

patricksnape commented Oct 27, 2014

I can confirm that this fixes the landmarkless image bug I was seeing. +1

nontas added a commit that referenced this pull request Oct 27, 2014

Merge pull request #494 from nontas/widgets_fix
Widgets bug fix and Active Template Model widget

@nontas nontas merged commit 90e577a into menpo:master Oct 27, 2014

2 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci The Travis CI build passed
Details

@jabooth jabooth removed the in progress label Oct 27, 2014

@nontas nontas deleted the nontas:widgets_fix branch Oct 27, 2014

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