On 13.03.2014 09:55, A. Stoewer wrote:
>
> In include/nix/DataTag.hpp:
>
> > @@ -60,15 +60,6 @@ class NIXAPI DataTag : virtual public base::IDataTag, public base::EntityWithSou
> >      }
> >
> >      /**
> > -     * Setter for the positions of a tag.
> > -     *
> > -     * @param position   DataArray.
> > -     */
> > -    void positions(const DataArray &positions) {
>
> I just noticed that not having this setters is specified here #144 
> <https://github.com/G-Node/pandora/pull/144>. However I do not 
> remember that we agreed on this.
> I always said, that the internal interface (e.g. IDataArray) should 
> have only the setter with the id. But for the front facing objects an 
> additional setter with the native type was IMO always intended. Not 
> having this setters also conflicts with the setter schama in 
> EntityWithMetadata where we have a setter |void metadata(const Section 
> &metadata)|.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/G-Node/pandora/pull/145/files#r10556459>.
>
I remember that we some time ago spoke about this, but thought the minds 
had changed... Actually, I am not sure about having two versions in the 
front facing objects. It is no problem to have both versions but then we 
would need them for _all_ methods working with entities and to me this 
appears bloated... any opinions?
(In the same context I then would like to point to issue #138 
<https://github.com/G-Node/pandora/issues/138>)