[Olena-patches] last-svn-commit-10-g749308a Various small fixes.

* doc/tutorial/tutorial.tex: Add more contacts. * mln/accu/label_used.hh: Do not use label<>::next(). * mln/draw/line.hh: Add a fixme. * mln/io/dump/save.hh: Close the file. * mln/io/plot/save.hh: Fix an invalid prototype. * mln/io/pnm/load.hh: Fix an invalid static check. * mln/io/raw/get_header.hh: Fix comment. * mln/labeling/pack.hh: Disable a static check on symbolic values. * mln/math/diff_abs.hh: Add a fixme. * mln/world/inter_pixel/full2image.hh: Use mln_concrete(). * mln/data/fill_with_value.hh, * mln/io/raw/load.hh, * mln/io/raw/save.hh, * mln/labeling/compute.hh, * mln/registration/icp.hh: Aesthetic changes. --- milena/ChangeLog | 30 ++++++++++++++++++++++++++++ milena/doc/tutorial/tutorial.tex | 4 +++ milena/mln/accu/label_used.hh | 5 ++- milena/mln/data/fill_with_value.hh | 3 +- milena/mln/draw/line.hh | 1 + milena/mln/io/dump/save.hh | 3 ++ milena/mln/io/plot/save.hh | 5 ++- milena/mln/io/pnm/load.hh | 3 +- milena/mln/io/raw/get_header.hh | 2 +- milena/mln/io/raw/load.hh | 2 +- milena/mln/io/raw/save.hh | 2 +- milena/mln/labeling/compute.hh | 10 ++++---- milena/mln/labeling/pack.hh | 2 +- milena/mln/math/diff_abs.hh | 5 +++- milena/mln/registration/icp.hh | 3 +- milena/mln/world/inter_pixel/full2image.hh | 4 +- 16 files changed, 65 insertions(+), 19 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index 7171949..04b9146 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,35 @@ 2010-02-16 Guillaume Lazzara &lt;z(a)lrde.epita.fr&gt; + Various small fixes. + + * doc/tutorial/tutorial.tex: Add more contacts. + + * mln/accu/label_used.hh: Do not use label<>::next(). + + * mln/draw/line.hh: Add a fixme. + + * mln/io/dump/save.hh: Close the file. + + * mln/io/plot/save.hh: Fix an invalid prototype. + + * mln/io/pnm/load.hh: Fix an invalid static check. + + * mln/io/raw/get_header.hh: Fix comment. + + * mln/labeling/pack.hh: Disable a static check on symbolic values. + + * mln/math/diff_abs.hh: Add a fixme. + + * mln/world/inter_pixel/full2image.hh: Use mln_concrete(). + + * mln/data/fill_with_value.hh, + * mln/io/raw/load.hh, + * mln/io/raw/save.hh, + * mln/labeling/compute.hh, + * mln/registration/icp.hh: Aesthetic changes. + +2010-02-16 Guillaume Lazzara &lt;z(a)lrde.epita.fr&gt; + * mln/topo/skeleton/crest.hh: Fix invalid PSI values for component edge sites. diff --git a/milena/doc/tutorial/tutorial.tex b/milena/doc/tutorial/tutorial.tex index 7ddbb60..5a61461 100644 --- a/milena/doc/tutorial/tutorial.tex +++ b/milena/doc/tutorial/tutorial.tex @@ -545,6 +545,10 @@ If you want to reach us directly, you can contact one of the following people: \begin{itemize} \item Thierry Geraud - Project Manager - thierry.geraud(a)lrde.epita.fr + \item Jonathan Fabrizio - jonathan.fabrizio(a)lrde.epita.fr + \item Yann Jacquelet - yann.jacquelet(a)lrde.epita.fr + \item Guillaume Lazzara - guillaume.lazzara(a)lrde.epita.fr + \item Roland Levillain - roland.levillain(a)lrde.epita.fr \end{itemize} diff --git a/milena/mln/accu/label_used.hh b/milena/mln/accu/label_used.hh index af82cd2..d6d2b20 100644 --- a/milena/mln/accu/label_used.hh +++ b/milena/mln/accu/label_used.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2009, 2010 EPITA Research and Development Laboratory +// (LRDE) // // This file is part of Olena. // @@ -116,7 +117,7 @@ namespace mln label_used<L>::take(const argument& l) { if (label_used_.size() <= l) - label_used_.resize(l.next(), false); + label_used_.resize(unsigned(l) + 1, false); label_used_(l) = true; } diff --git a/milena/mln/data/fill_with_value.hh b/milena/mln/data/fill_with_value.hh index 8ab0a66..5688ed1 100644 --- a/milena/mln/data/fill_with_value.hh +++ b/milena/mln/data/fill_with_value.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2008, 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2008, 2009 EPITA Research and Development Laboratory +// (LRDE) // // This file is part of Olena. // diff --git a/milena/mln/draw/line.hh b/milena/mln/draw/line.hh index f5cbffb..528dce3 100644 --- a/milena/mln/draw/line.hh +++ b/milena/mln/draw/line.hh @@ -30,6 +30,7 @@ /// \file /// /// Draw a line in an image. +/// \fixme Add specializations for horizontal lines (use pointers/memset). # include <mln/core/concept/image.hh> # include <mln/core/site_set/p_line2d.hh> diff --git a/milena/mln/io/dump/save.hh b/milena/mln/io/dump/save.hh index a361856..7043948 100644 --- a/milena/mln/io/dump/save.hh +++ b/milena/mln/io/dump/save.hh @@ -144,6 +144,9 @@ namespace mln internal::save_header(ima, file); internal::save_data(ima, file); + + file.close(); + trace::exiting("mln::io::dump::save"); } diff --git a/milena/mln/io/plot/save.hh b/milena/mln/io/plot/save.hh index 8010afa..9f38f4b 100644 --- a/milena/mln/io/plot/save.hh +++ b/milena/mln/io/plot/save.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2009, 2010 EPITA Research and Development Laboratory +// (LRDE) // // This file is part of Olena. // @@ -60,7 +61,7 @@ namespace mln template <typename T> void save(util::array<T>& arr, const std::string& filename, - int start_value = 0); + int start_value); # ifndef MLN_INCLUDE_ONLY diff --git a/milena/mln/io/pnm/load.hh b/milena/mln/io/pnm/load.hh index a48737a..46587df 100644 --- a/milena/mln/io/pnm/load.hh +++ b/milena/mln/io/pnm/load.hh @@ -197,7 +197,8 @@ namespace mln inline void load_raw_2d(std::ifstream& file, I& ima) { - if (sizeof(value::int_u8) == 1) + typedef mln_value(I) V; + if (sizeof(V) == 1) load_raw_2d_contiguous(file, ima); else load_raw_2d_uncontiguous(file, ima); diff --git a/milena/mln/io/raw/get_header.hh b/milena/mln/io/raw/get_header.hh index fa5dbf3..c263cff 100644 --- a/milena/mln/io/raw/get_header.hh +++ b/milena/mln/io/raw/get_header.hh @@ -28,7 +28,7 @@ /// \file /// -/// Load a Milena image rawed into a file. +/// Load a raw image header. # include <iostream> # include <fstream> diff --git a/milena/mln/io/raw/load.hh b/milena/mln/io/raw/load.hh index 7715740..57ca64c 100644 --- a/milena/mln/io/raw/load.hh +++ b/milena/mln/io/raw/load.hh @@ -1,4 +1,4 @@ -// Copyright (C) 2009, 2010 EPITA Research and Development Laboratory +// Copyright (C) 2010 EPITA Research and Development Laboratory // (LRDE) // // This file is part of Olena. diff --git a/milena/mln/io/raw/save.hh b/milena/mln/io/raw/save.hh index f526e57..8e927f7 100644 --- a/milena/mln/io/raw/save.hh +++ b/milena/mln/io/raw/save.hh @@ -1,4 +1,4 @@ -// Copyright (C) 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2010 EPITA Research and Development Laboratory (LRDE) // // This file is part of Olena. // diff --git a/milena/mln/labeling/compute.hh b/milena/mln/labeling/compute.hh index 0e74fd7..d29242f 100644 --- a/milena/mln/labeling/compute.hh +++ b/milena/mln/labeling/compute.hh @@ -38,11 +38,11 @@ /// and change 'input' from Image to Site_Set! /// ==> NO, see below. (Z) /// -/// The overload not taking 'input' as argument works on sites and would be a good -/// candidate for set::compute. However, the fact that this version of compute -/// is in the namespace 'labeling' means that it produces several results at the -/// same time for each labels which is not the case of other implementations of -/// compute. +/// The overload not taking 'input' as argument works on sites and +/// would be a good candidate for set::compute. However, the fact that +/// this version of compute is in the namespace 'labeling' means that +/// it produces several results at the same time for each labels which +/// is not the case of other implementations of compute. # include <mln/core/concept/image.hh> # include <mln/core/concept/accumulator.hh> diff --git a/milena/mln/labeling/pack.hh b/milena/mln/labeling/pack.hh index c5e766e..ebf9bb1 100644 --- a/milena/mln/labeling/pack.hh +++ b/milena/mln/labeling/pack.hh @@ -100,7 +100,7 @@ namespace mln pack_tests(const Image<I>& label, mln_value(I)& new_nlabels) { // FIXME: we may want to check that it is exactly a label. - mlc_is_a(mln_value(I), mln::value::Symbolic)::check(); + //mlc_is_a(mln_value(I), mln::value::Symbolic)::check(); mln_precondition(exact(label).is_valid()); (void) label; (void) new_nlabels; diff --git a/milena/mln/math/diff_abs.hh b/milena/mln/math/diff_abs.hh index f5559ff..e0d8edd 100644 --- a/milena/mln/math/diff_abs.hh +++ b/milena/mln/math/diff_abs.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2007, 2008, 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2007, 2008, 2009 EPITA Research and Development +// Laboratory (LRDE) // // This file is part of Olena. // @@ -29,6 +30,8 @@ /// \file /// /// Define diff_abs routine. +/// +/// \fixme Do we want a specific version for vectorial types? namespace mln diff --git a/milena/mln/registration/icp.hh b/milena/mln/registration/icp.hh index ac80fa3..13f9231 100644 --- a/milena/mln/registration/icp.hh +++ b/milena/mln/registration/icp.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2008, 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2008, 2009 EPITA Research and Development Laboratory +// (LRDE) // // This file is part of Olena. // diff --git a/milena/mln/world/inter_pixel/full2image.hh b/milena/mln/world/inter_pixel/full2image.hh index c861a10..db3bcf3 100644 --- a/milena/mln/world/inter_pixel/full2image.hh +++ b/milena/mln/world/inter_pixel/full2image.hh @@ -52,7 +52,7 @@ namespace mln /// \return A classical image without inter-pixel data. // template <typename I> - I + mln_concrete(I) full2image(const Image<I>& input); @@ -60,7 +60,7 @@ namespace mln template <typename I> - I + mln_concrete(I) full2image(const Image<I>& input_) { trace::entering("world::inter_pixel::full2image"); -- 1.5.6.5