(r16) PhotDM10RFC < IVOA

IVOA Web>PhotDM10RFC (revision 16) (raw view)~~EditAttach~~
---++ Photometry DM RFC

This document will act as *RFC* centre for the Photometry DM <a target="_blank" href="http://www.ivoa.net/Documents/PHOTDM/">http://www.ivoa.net/Documents/PHOTDM/</a>

This is the second RFC process for this DM. Previous RFC page can still be found at: http://wiki.ivoa.net/twiki/bin/view/IVOA/PhotDM10RFCOLD

Please notice this version is an updated version of the previous Photometry DM with some changes in the UML but maintaining the utypes as in the

current way. A parallel PhotDM draft is being prepared in parallel with the VO-DML utypes approach.

In order to add a comment to the document, please edit this page and add your comment to the list below in the format used for the example (include your TWiki.WikiName so authors can contact you for further information). When the author(s) of the document have considered the comment, they will provide a response after the comment.

Discussion about any of the comments or responses should be conducted on the [[mailto:dm@ivoa.net][dm@ivoa.net]]. However, please be sure to enter your initial comments here for full consideration in any future revisions of this document.
---++ Implementation details

   * VOSpec (ESAVO) (client)
   * Filter Profile Service (SVO) (Server + Client)
   * Catalogs with Photometry Data (CDS) (Server)
---+++ RFC Review Period: 04 July 2013 - 04 August 2013
---++ Comments from the IVOA Community during RFC period:
---+++ TCG Review Period: 05 August 2013 - 10 September 2013
---++ Comments from TCG member during the TCG Review Period:

WG chairs or vice chairs must read the Document, provide comments if any and formally indicate if they approve or not the Standard.

IG chairs or vice chairs are also encouraged to do the same, although their inputs are not compulsory.
---+++ TCG Chair & Vice Chair ( _Séverin Gaudet, Matthew Graham)_
---+++ Applications Working Group ( _Mark Taylor, Pierre Fernique)_

This is a clear and as far as I can tell well-motivated document. I don't have expert knowledge about photometry, so I can't comment on its adequacy for its intended purpose, but I don't see any problems with it from the Applications WG point of view, so we are happy to recommend acceptance.

There are a few typos and minor errors:

   * Acknowledgements: "initials versions" -&gt; "initial versions"
   * Sec 2: "as per explained" -&gt; "as explained"
OK to both -- IVOA.JesusSalgado - 2013-09-03

   * Sec 3: Figure 1 contains the box "Source" which doesn't seem to be doing anything - remove it?
Although this DM has not been developed, there are many IVOA parties that are asking for it so I would prefer to

mantain the placeholder. A comment has been added to clarify this -- IVOA.JesusSalgado - 2013-09-03

   * Sec 3.2.6: " =YYYY-MM-DD[T[hh[:mm[:ss[.s[TZD]]]]]]= " - I don't quite understand the "[TZD]" here. If TZD is supposed to be a time zone designator, the (unexplained, but fairly obvious) syntax suggests it is only permissible to use it if a fractional part of the seconds is explicitly included, which seems a bit strange. Also, I think I'm right in saying that general usage elsewhere in IVOA standards does not encourage/permit time zone indicators (e.g. DALI sec 3.1.2).
I agree TZD should be discouraged in VO context. Removed -- IVOA.JesusSalgado - 2013-09-03

   * Sec 3.2.8: "the more representative" -&gt; "the most representative"
OK -- IVOA.JesusSalgado - 2013-09-03

   * Sec 3.6.3: "See Appendix 5.2" - what document does this refer to? Not <nop>PhotDM itself.
Reference corrected -- IVOA.JesusSalgado - 2013-09-03

   * Appendix B: Table heading "UCD+1" should be "UCD 1+"?
   * Appendix B: "may belong to on of three" -&gt; "may belong to one of three"
   * Appendix B: "Transmission curve are also" -&gt; "Transmission curves are also"
   * Appendix B: "Other serialization using array of points" -&gt; "Other serializations using arrays of points"
   * Appendix C.1: "PhotometryFilter.identifer" -&gt; "PhotometryFilter.identifier"
   * Appendix C.1: ' =&lt;PARAM name="magtype" ... value="VEGmag" datatype="int"/&gt;= ' - the datatype appears to be wrong; also according to the table on p.35 the value should be "VEGAmag" not "VEGmag".
   * Appendix C.2: "service will contains some" -&gt; "service will contain some"
OK to all. Many thanks for the careful review -- IVOA.JesusSalgado - 2013-09-03

-- IVOA.MarkTaylor, IVOA.PierreFernique - 2013-07-18

---+++ Data Access Layer Working Group ( _Patrick Dowler, Francois Bonnarel)_
---+++ Data Model Working Group ( _Jesus Salgado, Omar Laurino_)

Approved-- IVOA.JesusSalgado - 2013-09-09
---+++ Grid & Web Service<em>s Working Group ( Andre Schaaff, Andreas Wicenec) </em>
Approved - IVOA.AndreSchaaff - 2013-09-09
---+++ Registry Working Group ( _Gretchen Greene, Pierre Le Sidaner_)
---+++ Semantics Working Group ( _Norman Gray, Mireille Louys_)
---+++ Da<em>ta Curation & Preservation Interest Group ( Alberto Accomazzi, Francoise Genova) </em>
---+++ _Knowledge Discovery in Databases Interest Group (_ _George Djorgovski_ _)_
---+++ Theory Interest Group ( _Franck Le Petit, Rick Wagner_)
---+++ Time Domain Interest Group (John Swinbank, Mike Fitzpatrick)
Topic revision: r16 - 2013-09-09 - AndreSchaaff
IVOA
Log in or Register
IVOA.net
Wiki Home
WebChanges
WebTopicList
WebStatistics
Twiki Meta & Help
IVOA
Know
Main
Sandbox
TWiki
TWiki intro
TWiki tutorial
User registration
Notify me
Working Groups
Interest Groups
Committees
Stds&Procs
www.ivoa.net
Documents
Events
Members
XML Schema