<div dir="ltr"><div dir="ltr"><div>All,<br><br></div>Got some additional feedback from Ada off-line which should be folded into the next iteration of the document.. see below:<br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 24, 2019 at 3:08 PM CresitelloDittmar, Mark &lt;<a href="mailto:mdittmar@cfa.harvard.edu">mdittmar@cfa.harvard.edu</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Ada,</div><div><br></div><div>Thank you for this input.. I&#39;ll address your points here, but also send a follow-up to the mail list on items which should go into the next iteration of the WD.</div><div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail-m_-488514180128521752gmail-m_6742760961863666400gmail-m_2732229572762887366gmail_attr">On Wed, Jan 23, 2019 at 11:42 AM AdaNebot &lt;<a href="mailto:ada.nebot@astro.unistra.fr" target="_blank">ada.nebot@astro.unistra.fr</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div>Hi Mark, </div><div><br></div><div>Thanks for your work putting all this together. Great job! </div><div><br></div><div>Since this model does concern time domain, I feel I should give some feedback… I could send some of my questions to the DM list but I would like to have an outside discussion with you too. Of course you are free to forward this email to anyone you consider necessary. Even reply in the DM list if you think that would bring something positive. I’ve put Arnold in CC. </div><div><br></div><div>First, I’m extremely sorry for my huge email, but I do want to get to understand some details. </div><div><br></div><div>So here are some comments in particular on section 6, package domain.time: </div><div>All the information written in section 6 package domain.time, page 26 was already shared by Arnold, back in 2017 during the Shanghai meeting:  <a href="http://mail.ivoa.net/pipermail/voevent/2017-May/003097.html" target="_blank">http://mail.ivoa.net/pipermail/voevent/2017-May/003097.html</a> and supported by the TDIG as stated in the TDIG twiki page: <a href="https://wiki.ivoa.net/twiki/bin/view/IVOA/DAL_DM_TDIG" target="_blank">https://wiki.ivoa.net/twiki/bin/view/IVOA/DAL_DM_TDIG</a></div><div>so +1 from my side. </div><div><br></div><div>Point 6 end of the paragraph: “I assume we will be sticking with TT and TDB…” —&gt; Remove that sentence from the text. I understand this is not a final version, but still no point in leaving it there.</div></div></blockquote><div><br></div><div>You are the second to mention that this.</div><div>  <span style="color:rgb(255,0,0)">ACTION</span>:  I&#39;ll remove the assumption sentence, and leave this bullet with the TCG,TCB,TDB text and &quot;More in the cited A&amp;A paper&quot;</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><br></div><div><b>Section 6.1.1. TimeFrame.refPosition </b></div><div>type: coords.domain.space.RefLocation “The spatial location at which the coordinate is considered to have been take from”. </div><div>Shouldn&#39;t there be a pointer to a vocabulary here as it is done for the timescale? I’m thinking here GEOCENTER, BARYCENTER,… and what has already been discussed in other emails. </div><div>Also, a name can be a location but indicating (long,lat,height) is also a location but with a completely different format. I would expect this to be more specific and mention the kind of format this should have. If this is meant to be free for interpretation, then I think it should be stated like that. </div></div></blockquote><div><br></div><div>The RefLocation type is abstract, and has both StdRefLocation (vocabulary) and CustomRefLocation subtypes.  I tend to not re-state the options at places where abstract types are used, unless there is something specific about that usage worth stating.  Since RefLocation is used both here and in refDirection, it might be useful to elaborate here.</div><div>   <span style="color:rgb(255,0,0)">ACTION</span>: add text to description..</div><div>   &quot;In this model, RefLocation supports locations provided as either a standard reference position (GEOCENTER) or a coordinate specifying a custom location (eg: long, lat, height).&quot;</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><br></div><div><b>Section 6.1.2. TimeFrame.timescale</b></div><div><a href="https://ivoa.net/vocabularies/coords/TimeScale" target="_blank">https://ivoa.net/vocabularies/coords/TimeScale</a></div><div>I couldn’t find the link, but I guess this will be existing soon?  </div></div></blockquote><div><br></div><div>This is a work in progress.  I&#39;ve sent the lists to Markus/Mireille and he installed them in volute back in mid-Dec.</div><div>He had some comments on the content of some of them (the space reference frames had some hierarchy issue).  The action is on me to get back to that and correct the lists and &#39;build&#39; them(?) to some location which that URL would point to.  </div><div><br></div><div>excerpt from my last mail with Markus/Mireille on this:</div><div>&quot;  * once these are established vocabularies, the content becomes a separate topic (IMO), so <br></div><div><div>     am happy to have things cut which are not immediately useful to the community<br></div><div>    o with these in place, the next step would be to point SSIG and TDIG at them to make recommendations/additions&quot;</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><br></div><div><b>Section 6.1.3. TimeFrame.refDirection</b></div><div>So here you mean the coordinates of the observed object(s)? This is needed for light travel corrections between two different observing points (refPosition) and also for combining observations taken at two different moments where the Earth (or observatory position) was at a different location and therefore the solar system ephemerides is needed. I think it cannot harm having things clearly defined and explained to some level. Maybe this is not the best phrasing but in those lines. </div></div></blockquote><div><br></div><div><span style="color:rgb(255,0,0)">ACTION</span>: I&#39;ll see about clarifying the text for how/when this is used.  Arnold.. any suggestions?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><b>Section 6.2.</b> <b>TimeStamp</b><br></div><div>As I understand the UML —&gt; Section 6.2 defining the TimeStamp should be made clearer, indicating that a TimeStamps can be either a TimeInstant or a TimeOffset (real quantity with respect to a TimeInstant) and that TimeInstants have to be given in JD, MJD or ISO-8601. This is very clear to me when looking at the UML, but not so in the text. Is there a reason why there’s a check-box on Fig.6 on CoordFrame ? </div></div></blockquote><div><br></div><div>* again, this is an efficiency thing.. not stating the current extensions as part of the parent description, which makes more text which can get out of sync.  In my other model docs, I&#39;ve had more pictures (small snippets of families such as this).  That would show the group at the point of description.  With this ivoatex doc, I&#39;ve limited it to 1 diagram per section, and in this particular case, that diagram would be about 80% of what is in Fig. 6, just 2 pages earlier.. so I&#39;m kind-of on the fence here about changing this.</div><div><br></div><div>* There is no check-box in Fig 6... I think sometimes different PDF readers do funky things.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><b><br></b></div><div><b>Section 6.4. ISOTime:</b> </div><div>“ISO-8601 standard within the <u>restrictions imposed by the IVOA”</u> What are those? Removing the time-zone? I would suggest to add the restriction and properly reference it. </div><div><br></div></div></blockquote><div>Ah.. That is described in the appendices (E.1.2).  That is a bit of a loose end.. I transferred that text from the cube model, but this should be consistent with what is in the DALI Rec.  DALI goes into more detail than what is here (which is appropriate for a Data Access standard vs a model), but are consistent.  The DALI Rec states that it follows the conventions used by FITS (2001) and STC (2007).</div><div><br></div><div><span style="color:rgb(255,0,0)">ACTION:</span>  I think the thing to do here is bring the text from E.1.2. into Section 6.4, and review the details w.r.t. DALI Section 3.3.3.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div></div><div><b>Section 6.6. MJD:</b> </div><div>Typo in page 28: T(MJD) = T(JD) - 2440000.5 —&gt;  T(MJD) = T(JD) - 2400000.5 </div><div><br></div></div></blockquote><div><br></div><div><span style="color:rgb(255,0,0)">ACTION:</span> correct this.</div><div> <br></div></div></div></div></div></div></div></div></div></div></div></div></blockquote><div><br></div>There are additional questions regarding representing a particular scenario which will feed into the other actions on describing the supported cases, corresponding derived requirements, and example files.<br></div><div class="gmail_quote">Will follow-up as that solidifies.<br><br></div><div class="gmail_quote">Mark<br><br></div><div class="gmail_quote"><br><br><br></div></div>