14 Apr
2004
14 Apr
'04
12:10 a.m.
Index: olena/oln/morpho/geodesic_dilation.hh
--- olena/oln/morpho/geodesic_dilation.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g
(oln/37_geodesic_d 1.11.1.12 600)
+++ olena/oln/morpho/geodesic_dilation.hh Tue, 13 Apr 2004 23:03:36 +0200 thivol_d
(oln/37_geodesic_d 1.11.1.12 600)
@@ -69,7 +69,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** save(oln::morpho::geodesic_dilation(im2, im1, oln::neighb_c4()),
** IMG_OUT
"oln_morpho_geodesic_dilation.pbm");
@@ -130,7 +130,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** save(oln::morpho::sure::geodesic_dilation(im2, im1, oln::neighb_c4()),
** IMG_OUT
"oln_morpho_sure_geodesic_dilation.pbm");
Index: olena/oln/morpho/geodesic_erosion.hh
--- olena/oln/morpho/geodesic_erosion.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g
(oln/36_geodesic_e 1.19 600)
+++ olena/oln/morpho/geodesic_erosion.hh Tue, 13 Apr 2004 23:03:01 +0200 thivol_d
(oln/36_geodesic_e 1.19 600)
@@ -67,7 +67,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** save(oln::morpho::geodesic_erosion(im1, im2, oln::neighb_c4()),
** IMG_OUT
"oln_morpho_geodesic_erosion.pbm");
@@ -128,7 +128,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** save(oln::morpho::sure::geodesic_erosion(im1, im2, oln::neighb_c4()),
** IMG_OUT
"oln_morpho_sure_geodesic_erosion.pbm");
Index: olena/oln/morpho/reconstruction.hh
--- olena/oln/morpho/reconstruction.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g
(oln/35_reconstruc 1.13.1.13 600)
+++ olena/oln/morpho/reconstruction.hh Tue, 13 Apr 2004 23:02:25 +0200 thivol_d
(oln/35_reconstruc 1.13.1.13 600)
@@ -74,7 +74,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::sure::geodesic_reconstruction_dilation(im2,
** im1,
@@ -146,7 +146,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::sequential::geodesic_reconstruction_dilation(im2,
** im1,
@@ -262,7 +262,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::hybrid::geodesic_reconstruction_dilation(im2,
** im1,
@@ -367,7 +367,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::sure::geodesic_reconstruction_erosion(im1,
** im2,
@@ -440,7 +440,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::sequential::geodesic_reconstruction_erosion(im1,
** im2,
@@ -552,7 +552,7 @@
** typedef oln::image2d<ntg::int_u8> im_type;
**
** im_type im1(oln::load(IMG_IN "lena128.pgm"));
- ** im_type im2(oln::morpho::opening(im1, oln::win_c4p()));
+ ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p());
**
** oln::save(oln::morpho::hybrid::geodesic_reconstruction_erosion(im1,
** im2,
Index: olena/oln/topo/tarjan/flat-zone.hh
--- olena/oln/topo/tarjan/flat-zone.hh Fri, 02 Apr 2004 16:53:54 +0200 palma_g
(oln/r/35_flat-zone. 1.13 600)
+++ olena/oln/topo/tarjan/flat-zone.hh Wed, 14 Apr 2004 00:03:57 +0200 thivol_d
(oln/r/35_flat-zone. 1.13 600)
@@ -88,7 +88,7 @@
typedef abstract::tarjan_with_attr<exact_type> super_type; ///< Type of parent
class.
friend class super_type;
- friend class super_type::super_type;
+ friend class abstract::tarjan<exact_type>;
/*!
** \brief Constructor.
**
--
Damien Thivolle
damien.thivolle(a)lrde.epita.fr
1:42 p.m.
Akim Demaille wrote:
Index: olena/ChangeLog
from Damien Thivolle <damien(a)lrde.epita.fr>
* olena/oln/morpho/geodesic_dilation.hh: Fix to make compile with ICC.
* olena/oln/morpho/geodesic_erosion.hh: Likewise.
* olena/oln/morpho/reconstruction.hh: Likewise.
* olena/oln/topo/tarjan/flat-zone.hh: Likewise.
>>"Damien" == Damien Thivolle <damien(a)lrde.epita.fr> writes:
Ce patch est sans ChangeLog, ni justification : *pourquoi* ICC demande
ce changement ???
1) img_type out (algo(in));
2) img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains
algos. L'écriture (2) fonctionne toujours et doit
toujours étre utilisée.
La raison est certainement compliquée. Le problème vient du
constructeur par copie des images qui n'est pas implémenté,
mais qui est appelé par ICC certainement à cause de problèmes
de constitude.
Nous avons cherché comment réparer et non pourquoi c'est cassé.
J'attends d'avoir le temps de comprendre le pourquoi.
--
Niels
4:42 p.m.
>> "Niels" == Niels Van Vliet
<niels(a)lrde.epita.fr> writes:
Akim Demaille wrote:
>>>> "Damien" == Damien Thivolle <damien(a)lrde.epita.fr>
writes:
> Ce patch est sans ChangeLog,
Index: olena/ChangeLog
from Damien Thivolle <damien(a)lrde.epita.fr>
* olena/oln/morpho/geodesic_dilation.hh: Fix to make
compile with ICC.
* olena/oln/morpho/geodesic_erosion.hh: Likewise.
* olena/oln/morpho/reconstruction.hh: Likewise.
* olena/oln/topo/tarjan/flat-zone.hh: Likewise.
> ni justification : *pourquoi* ICC demande
> ce changement ???
1) img_type out (algo(in));
2) img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains
algos. L'écriture (2) fonctionne toujours et doit
toujours étre utilisée.
La raison est certainement compliquée. Le problème
vient du
constructeur par copie des images qui n'est pas implémenté,
mais qui est appelé par ICC certainement à cause de problèmes
de constitude.
Nous avons cherché comment réparer et non pourquoi
c'est cassé.
J'attends d'avoir le temps de comprendre le pourquoi.
OK.
Est-ce bien noté qqpart ?
5:58 p.m.
Akim Demaille wrote:
c'est dû aux bizarreries qu'on doit écrire pour
gérer la constance des images ; les détails sont lisibles ici :
http://www.lrde.epita.fr/cgi-bin/twiki/view/Projects/OlenaConstImages
"Niels" == Niels Van Vliet <niels(a)lrde.epita.fr> writes:
[...]
1) img_type out (algo(in));
2) img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour
certains
algos. L'écriture (2) fonctionne toujours et doit
toujours étre utilisée.
[...]
OK.
Est-ce bien noté qqpart ?
20 Apr
20 Apr
9:06 a.m.
>> "Théo" == Thierry Géraud
<thierry.geraud(a)lrde.epita.fr> writes:
Akim Demaille wrote:
> > "Niels" == Niels Van Vliet <niels(a)lrde.epita.fr> writes:
> > [...]
> > 1) img_type out (algo(in));
> > 2) img_type out = algo(in);
> > L'écriture (1) ne fonctionne pas pour certains
> > algos. L'écriture (2) fonctionne toujours et doit
> > toujours étre utilisée.
> > [...]
> OK.
> Est-ce bien noté qqpart ?
c'est dû aux bizarreries qu'on doit écrire
pour
gérer la constance des images ; les détails sont lisibles ici :
Alors le ChangeLog devrait clairement pointer dessus !
7742
days inactive
7749
days old
6 comments
4 participants
participants (4)
-
Akim Demaille -
Damien Thivolle -
Niels Van Vliet -
Thierry Géraud