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

New tool 2dCompImage #313

Merged
merged 8 commits into from
Jan 29, 2018
Merged

New tool 2dCompImage #313

merged 8 commits into from
Jan 29, 2018

Conversation

kerautret
Copy link
Member

@kerautret kerautret commented Jan 12, 2018

PR Description

New tools to compare 2D images (and visualize errors).

Checklist

  • Doxygen documentation of the code completed (classes, methods, types, members...).
  • Main tool doxygen documentation (following existing documentation of DGtalTools documentation.
  • Check if it follows the tools structure described in CONTRIBUTING.md
  • New entry in the ChangeLog.md added.
  • Update the readme with potentially a screenshot of the tools if it applies.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).

@kerautret kerautret changed the title 2d comp image New tool 2dCompImage Jan 14, 2018
@@ -19,7 +19,7 @@
@section visualization_Doc Visualization



- @ref Doc2dCompImage : Computes and displays image comparisons (squared and absolute differences).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compare images and display differences (squared and absolute differences).

?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes better thanks ;)

-e [ --imageError ] arg Output error image basename (will generate
two images <basename>MSE.ppm and
<basename>MAE.ppm).
-S [ --fixMaxColorValueMSE ] arg Fix the maximal color value for the scale
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix -> set?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

getMAEstats(const Image2D & imageA, const Image2D &imageB, Image2DErr &imageMAE)
{
Statistic< int> stat(false);
for(Image2D::Domain::ConstIterator it = imageA.domain().begin();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or more simply:

for(auto point: imageA.domain())

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

;)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes thanks C++11 ;)

("imageA,a", po::value<std::string >(), "Input filename of image A." )
("imageB,b", po::value<std::string >(), "Input filename of image B." )
("imageError,e", po::value<std::string >(), "Output error image basename (will generate two images <basename>MSE.ppm and <basename>MAE.ppm)." )
("fixMaxColorValueMSE,S", po::value<int>(), "Fix the maximal color value for the scale display of MSE (else the scale is set the maximal MSE value)." )
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix -> set ?

@kerautret
Copy link
Member Author

all done @dcoeurjo thanks for the review

@dcoeurjo
Copy link
Member

perfect. thanks.

@dcoeurjo dcoeurjo merged commit 9827da5 into DGtal-team:master Jan 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants