- From: Jim Barnett <Jim.Barnett@genesyslab.com>
- Date: Wed, 11 Nov 2009 07:20:53 -0800
- To: "Greg Beauchesne" <gbeauch1@jhu.edu>, "VoiceBrowser" <www-voice@w3.org>
Greg, We will reply to the rest of your comments as we have time to review them. Here are responses to a few of them: 5. Section 9 on IDs does not specify the scope of uniqueness >> The section states: "The values of all attributes of type "id" MUST be unique within the session." 7. The algorithm seems to potentially run into infinite recursion on default history state transitions if multiple default history state transition targets create a circuit >> We will look at this more closely, but in general SCXML and all other state machine languages are so rich in possibilities for infinite loops that we can never hope to eliminate all of them. And from your email of Nov 11 1. In section 3.5.2, where it says "the 'target' of the transition must be a descendant of the <initial> element's parent <state>", was it actually intended to say "descendant" and not "child" here? I ask because if it is actually supposed to be "descendant", then there are some issues that may need to be addressed regarding <initial> and <parallel>. I can elaborate, but I want to make sure this is the case first. >> yes, the intent was 'descendant', so we would like to hear about the issues with <initial> and <parallel> 4. What are the semantics of the in() predicate in any of the intermediate places where executable content is used? What I mean by this is that just about everywhere that executable content is used except for transition/@cond, the current state configuration is in the middle of being modified (onentry, onexit, (state | parallel)/transition, initial/transition). If the answer here is "add/remove states in the configuration when the algorithm says so," then OK. I just wanted to make sure, since 8.1, 10.2.5.1, and 10.3.4.1 don't say anything about this. >> Yes, the intent is to add/remove states following the algorithm. 9. Are there any plans for state machine instance serialization? i.e. I want to take a running instance, save its configuration, history states, event queue, etc. to a byte array, and then restore it at a later time >> We have discussed this and think it's out of scope, at least for the first version of the spec. We expect that most implementations will support something like this, and are interested in suggestions for things that we could add to the spec to make serialization easier, but we don't see the need for a standard serialization format at the moment. = Jim Barnett -----Original Message----- From: www-voice-request@w3.org [mailto:www-voice-request@w3.org] On Behalf Of Greg Beauchesne Sent: Sunday, November 01, 2009 10:02 PM To: VoiceBrowser Subject: Re: [scxml] sixth WD of SCXML is published Kazuyuki Ashimura wrote: > Hi www-voice, > > The sixth Working Draft of "State Chart XML (SCXML): State Machine > Notation for Control Abstraction" has been published as follows. Hello. My name is Greg Beauchesne. I guess I'm new here, but I've been working on an SCXML implementation of my own and I figured that means I should probably get in on this list. So, without further ado... 1. I'm glad that the whole event name prefix issue has been updated to use tokens. This had been causing me issues with event names like "up" vs. "update". 2. Would it be possible for transition/@event to be of type NMTOKENS instead of NMTOKEN? As it stands, if I want to have the same transition be taken for two completely different events, with all of the associated executable content, I have to copy and paste the entire transition definition or otherwise use scripted functions to achieve common functionality. I think this would not break anything, since NMTOKEN already forbids whitespace. (Although technically I guess it's not NMTOKEN anyway, according to the schema in D.1) 3. I also like the new expr vs. non-expr attributes; this really clarifies when a string or expression should be used. On <send> and <invoke>, I would say that @id and @idlocation is inconsistent with the rest (@idexpr instead of @idlocation would probably match better), but I realize this is probably for compatibility with previous WDs, not to mention that it is the one attribute that acts as an lvalue. 4. Along the lines of the expr/non-expr attributes, I would see value in something similar for <log>. It seems to me that @label is not intended to be replacement for @expr (i.e. <log label="test"/> is not necessarily the same as <log expr="'test'"/>, especially since @expr is required and @label is not). So maybe something like <log message="My message"/>? If that were the case, though, it would probably get confusing as to whether to use @label or @message. What was the intent behind the @label attribute? Alternatively or in addition, I think <log> could support child content (at least text nodes) as an alternative to either the @label or @expr attributes. <assign> and <data> already do this. For example, this: <log>A thing happened</log> ....would be the same as this (assuming a compatible scripting language): <log expr="'A thing happened'"/> 5. Section 9 on IDs does not specify the scope of uniqueness. I know that the XML ID type specifies that it is unique throughout the document, but it seems like states/sends/invokes/data model fields would all be in different namespaces (not to be confused with XML namespaces). I would think it to be pretty useful for an <invoke> or a <send> to share the ID of the <state> that contained it. If the intent was to forbid this sort of thing, I think that should be explicitly stated. 6. Is the default transition on history states permitted to have executable content? The algorithm seems to say no (the transition is just used as though it were the history value), while section 3.11.2 does not say. If the answer really is "no", is there any reason to have a <transition> child as opposed to just an attribute with the default targets, other than consistency? (i.e. <history default="blah"/> seems just as good as <history><transition target="blah"/></history>) 7. The algorithm seems to potentially run into infinite recursion on default history state transitions if multiple default history state transition targets create a circuit (i.e. it targets either itself or another history state that then targets the original history state). A potential solution (assuming no executable content from #6 above) would be to simply forbid the default history state transition from having a sibling history state as a target. There is no reason this would be necessary, since A) before the history values are assigned, setting the target of H1 to H2 would have the same result as setting the target of H1 to the target of H2, and B) all sibling history state values are assigned at the same time, which eliminates the default transitions -- there is never a time when one sibling history will be using its default transition and the other will not. ------------------- Lastly, here are some proofreading errors: Section 3.2.2: Last paragraph -- in the text '...an "initial" attribute or am <initial> element...', I'm assuming "or am <initial>" should be "or an <initial>". Sections 4.1.6, 4.3.3, E.1, E.7, G.1: The examples don't appear to have been updated for the new expr and non-expr attributes. Section 10.3.3.1: The XPath in the <assign> is incorrect. "city[id='nyc']" should be "city[@id='nyc']". Section B: addStatesToEnter() uses the "LCA" variable in its history state handling, but it's never actually assigned a value. (I expect this is from being moved from enterStates() in the previous WD, which means that a likely correction is "root".) Section C: scxml/@initialstate in the example should be scxml/@initial. ------------------- Those are my comments for now. Thank you for your work on SCXML. -- Greg ------------------------------------------------------------------------------------------------------------------- CONFIDENTIALITY NOTICE: This e-mail and any files attached may contain confidential and proprietary information of Alcatel-Lucent and/or its affiliated entities. Access by the intended recipient only is authorized. Any liability arising from any party acting, or refraining from acting, on any information contained in this e-mail is hereby excluded. If you are not the intended recipient, please notify the sender immediately, destroy the original transmission and its attachments and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Copyright in this e-mail and any attachments belongs to Alcatel-Lucent and/or its affiliated entities.
Received on Wednesday, 11 November 2009 21:16:52 UTC