Il faut remettre la vielle fonction et expliquer les différences entre les deux.
Index: olena/ChangeLog from Niels Van Vliet niels@lrde.epita.fr
* olena/oln/morpho/attributes.hh (integral): Change the function.
Index: olena/oln/morpho/attributes.hh --- olena/oln/morpho/attributes.hh Fri, 14 May 2004 20:29:58 +0200 van-vl_n (oln/j/45_attributes 1.13.1.1 600) +++ olena/oln/morpho/attributes.hh Sat, 15 May 2004 19:15:24 +0200 van-vl_n (oln/j/45_attributes 1.13.1.1 600) @@ -353,6 +353,7 @@ ** \brief Integral attribute. ** ** It is equivalent to volume in 2D, and weight in 3D. + ** FIXME: Add math definition here. */ template <class T = unsigned, class Exact = mlc::final> class integral_type: @@ -386,8 +387,6 @@ integral_type(const abstract::image<I> &input, const oln_point_type(I) &p, const env_type &) : - reflevel_(input[p]), - area_(ntg_unit_val(value_type)), value_(input[p]) { }; @@ -404,26 +403,6 @@ mlc_dispatch(getValue)(); };
- /*! - ** \brief Accessor to the reference level. - ** - ** \see getReflevel_impl() - */ - const value_type &getReflevel() const - { - mlc_dispatch(getReflevel)(); - }; - - /*! - ** \brief Accessor to the current area. - ** - ** \see getArea_impl() - */ - const value_type &getArea() const - { - mlc_dispatch(getArea)(); - }; - // impl part /*! ** \brief Implementation of getValue(). @@ -438,31 +417,6 @@ return value_; };
- /*! - ** \brief Implementation of getReflevel(). - ** - ** Override this method in order to provide a new version of - ** getReflevel(). - ** - ** \warning Do not call this method, use getReflevel() instead. - */ - const value_type &getReflevel_impl() const - { - return reflevel_; - }; - - /*! - ** \brief Implementation of getArea(). - ** - ** Override this method in order to provide a new version of - ** getArea(). - ** - ** \warning Do not call this method, use getArea() instead. - */ - const value_type &getArea_impl() const - { - return area_; - };
/*! ** \brief += operator implementation. @@ -473,8 +427,7 @@ */ void pe_impl(const self_type &rhs) { - value_ += rhs.getValue() + area_ * tools::diffabs(reflevel_, rhs.getReflevel()); - area_ += rhs.getArea(); + value_ += rhs.getValue(); };
/*! @@ -502,8 +455,6 @@ };
protected: - value_type reflevel_; ///< Reference level. - value_type area_; ///< Current area. value_type value_; ///< Current value (deduced from area and level). };
Niels van Vliet van-vl_n@lrde.epita.fr writes:
Il faut remettre la vielle fonction et expliquer les différences entre les deux.
Je vais faire ca apres le seminaire.
[...]
ok.
"Niels" == Niels van Vliet van-vl_n@lrde.epita.fr writes:
Il faut remettre la vielle fonction et expliquer les différences entre les deux.
Index: olena/ChangeLog from Niels Van Vliet niels@lrde.epita.fr
- olena/oln/morpho/attributes.hh (integral): Change the function.
ce ChangeLog ne correspond pas a ton changement ! TOn changement c'est avant tout de la suppression (et le ChangeLog ne le montre pas). Par ailleurs, le ChangeLog explique le changement et ses motivations. Ici tu ne reportes que "changement"... C'est un peu court pour les suivants.
Akim Demaille wrote:
"Niels" == Niels van Vliet van-vl_n@lrde.epita.fr writes:
Il faut remettre la vielle fonction et expliquer les différences entre les deux.
Index: olena/ChangeLog from Niels Van Vliet niels@lrde.epita.fr
- olena/oln/morpho/attributes.hh (integral): Change the function.
ce ChangeLog ne correspond pas a ton changement ! TOn changement c'est avant tout de la suppression (et le ChangeLog ne le montre pas). Par ailleurs, le ChangeLog explique le changement et ses motivations. Ici tu ne reportes que "changement"... C'est un peu court pour les suivants.
Oui, je n'étais pas sur de moi. L'intégral ne donnait pas de résultat correcte. C'est pourquoi je l'ai corrigé.
Or je n'ai pas utilisé le mot corrigé. En effet j'ai vu que c'était compliqué, et que le type qui l'avait écrit avait du réfléchir. Ce que j'ai corrigé n'était pas un bug, mais un algo qui n'avait pas le bon nom. Je vais l'ajouter et mettre de la doc dès que j'ai le temps de comprendre ce qu'il fait.