* 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 <z(a)lrde.epita.fr>
+ 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 <z(a)lrde.epita.fr>
+
* 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