Hi Andre
Thanks for your feedback. The document has been updated http://hepunx.rl.ac.uk/egee/sa3-uk/sd/ISNSpec.pdf
Regards
Antony
> - Use cases: the document is not very explicit about the
> requirements, and only vaguely refers to the SAGA use cases - I think
> the requirements should be defined in somewhat more detail, and/or one
> or more use cases should explicitly be mentioned.
Now mentions UC 1, section 7
>
>
> - "No use-case has been identified for the operators >=, >, <=, < to
> be applied to
> strings. An Implementation wishing to support these comparison
> operators on strings
> MUST select a collation sequence. Alternatively, an implementation CAN
> treat all
> string comparisons as true, or reject them as invalid SQL."
>
> IMHO, if the operators are defined, they should be evaluated, or an
> error should be thrown. I think it will 'surprise' users if the ops
> are accepted, but not applied.
I am inclined to agree but this is consistent with how the SD spec describes the treatment of strings
>
>
> - the classes do not inherit the saga::async interface, and thus will
> not have asynchronous operations. Is that on purpose, or an
> oversight?
Again this is to remain consistent with the SD spec
>
>
> - "model: the name of the information model"
>
> That seems to be a freeform string? I think it would be useful to
> at least propose (SHOULD) string constants for glue 1 and 2?
A note added
>
>
> - "in session"
>
> by convention in the core spec, the session parameter should always
> be the first one. It is still optional though - see core spec for
> details (basically, it is up to he language binding how to render
> that).
Updated
>
>
> - You do not have TimeOut, PermissionDenied,
> Authorization/Authentication etc exceptions. I am unclear about the
> underlying protocol (and the API should not care), but are you sure
> those error modes do not (ever) apply to the calls?
>
>
> - some calls have no exceptions whatsoever - are you sure they will
> *always* succeed? :-)
Added to constructor
get_data and list_related_entity_names
the data has already been loaded and parsed as a result of the constructor so will always succeed
exceptions Timeout to get_related_entities
these calls should return objects that you are authorized to see
>
>
> - what is "N.B" in the notes for "list_related_entity_names" ?
Removed
>
>
> - indentation seems sometimes faulty, in particular on line breaks in
> the verbatim sections, but also in the table in 3.1
Fixed
>
>
> - in the example, the include should just be <saga/saga.hpp> - at
> least we don't define individual includes for any other package so
> far.
Changed
>
>
> - you list two prototypes of 'get_related_entities' in the IDL
> section, but only one set of details later on. I guess you could use
> a default parameter value for the filter in IDL to fix this?
Added comment saying that the operation is overloaded, this is then consistent with the SD spec
>
>
>
> Hope that helps,
>
> Andre.
>
>
> On Sat, Feb 26, 2011 at 1:03 PM, Andre Merzky <andre(a)merzky.net> wrote:
> > Dear SAGA group,
> >
> > on behalf of Steve Fisher and Antony Wilson, please consider the
> > attached document. If there are no objections raised on this list
> > within the next 10 days, we intent to submit this document to the OGF
> > editor for publication as P-REC.
> >
> > Thanks to Steve and Antony for working on this document! :-)
> >
> > Best, Andre.
> >
> >
> > ---------- Forwarded message ----------
> > From: Steve Fisher <dr.s.m.fisher(a)gmail.com>
> > Date: Fri, Feb 25, 2011 at 11:36 PM
> > Subject: ISN spec
> > To: Shantenu Jha <sjha(a)cct.lsu.edu>, Andre Merzky <andre(a)merzky.net>,
> > Thilo Kielmann <kielmann(a)cs.vu.nl>
> > Cc: Antony Wilson <Antony.Wilson(a)stfc.ac.uk>
> >
> >
> > Hi,
> >
> > Antony made the few small changes as requested at the last OGF and we
> > would now like to circulate it for a final call within the group.
> >
> > If you agree please do so - then after a suitable period it can go to
> > the OGF editor.
> >
> > It is at http://hepunx.rl.ac.uk/egee/sa3-uk/sd/ISNSpec.pdf
> >
> > Steve
> >
> >
> >
> > --
> > So much time, so little to do...
> > [Garfield]
> >
>
>
>
> --
> So much time, so little to do...
> [Garfield]
> --
> saga-rg mailing list
> saga-rg(a)ogf.org
> http://www.ogf.org/mailman/listinfo/saga-rg
--
Scanned by iCritical.
Hi all,
we have been asked to register session for OGF-31 - thus a quick poll
who intents to attend. I realize that the travel effort is
significant, and, depending on your location, possibly costly. OTOH,
the registration fees will be very low this time (~100,--$, see
http://ogf.org/OGF31/registration.php) and it will be a good
opportunity to synchronize with the API efforts. Plus the usual
pleasure of having SAGA sessions of course :-)
So, please let me know if you plan to attend (or plan to not attend) OGF-31.
Best, Andre.
--
Nothing is ever easy...
Dear SAGA group,
on behalf of Steve Fisher and Antony Wilson, please consider the
attached document. If there are no objections raised on this list
within the next 10 days, we intent to submit this document to the OGF
editor for publication as P-REC.
Thanks to Steve and Antony for working on this document! :-)
Best, Andre.
---------- Forwarded message ----------
From: Steve Fisher <dr.s.m.fisher(a)gmail.com>
Date: Fri, Feb 25, 2011 at 11:36 PM
Subject: ISN spec
To: Shantenu Jha <sjha(a)cct.lsu.edu>, Andre Merzky <andre(a)merzky.net>,
Thilo Kielmann <kielmann(a)cs.vu.nl>
Cc: Antony Wilson <Antony.Wilson(a)stfc.ac.uk>
Hi,
Antony made the few small changes as requested at the last OGF and we
would now like to circulate it for a final call within the group.
If you agree please do so - then after a suitable period it can go to
the OGF editor.
It is at http://hepunx.rl.ac.uk/egee/sa3-uk/sd/ISNSpec.pdf
Steve
--
So much time, so little to do...
[Garfield]
