${taskforce.name} Avatar
  1. OMG Task Force

Robotic Interaction Service Framework 1.1 RTF — All Issues

  • Key: ROIS11
  • Issues Count: 22
Open Closed All
All Issues

Issues Summary

Key Issue Reported Fixed Disposition Status
ROIS11-29 overview Package and UML Class diagram. RoIS 1.0 RoIS 1.1 Deferred closed
ROIS11-28 "Overview" section is a bit misplaced. RoIS 1.0 RoIS 1.1 Deferred closed
ROIS11-3 Atomic allocation of multiple HRI Components required RoIS 1.0b1 RoIS 1.1 Deferred closed
ROIS11-2 Ambiguous definition of "Condition" RoIS 1.0b1 RoIS 1.1 Deferred closed
ROIS11-1 Dependency among HRI Components should be defined RoIS 1.0b1 RoIS 1.1 Deferred closed
ROIS11-23 CORBA IDL needs to be corrected to be conformant with CORBA standard RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-22 C++ headers need to be corrected RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-4 "operable time" in System Information should be modified to deal with multiple time spans RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-21 Change cited version of RLS from 1.0 to 1.1 RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-6 Definition of timestamp refers to non normative document [W3C-DT]. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-40 CORBA PSM fails to compile. It should implement data types imported from RLS 1.1. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-39 C++ PSM should implement data types imported from RLS 1.1. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-41 Add an organization to Acknowledgment. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-32 Component Profiles are not valid against the XML Schema dtc/15-06-11.xsd. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-31 C++/CORBA PSMs lack Include protection. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-30 Empty comments in XML files should be removed. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-27 Reference to UML specification should be updated from 2.3 to 2.4.1. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-19 XML Profiles of Basic Component is missing. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-16 Proposal to add a new example of WheelChairComponent as C.4. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-13 DateTime in User-defined HRI Component should refer to [ISO8601]. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-11 inconsistent use of vector and List in C++ PSM. RoIS 1.0 RoIS 1.1 Resolved closed
ROIS11-9 Description of a parameter is missing. RoIS 1.0 RoIS 1.1 Resolved closed

Issues Descriptions

overview Package and UML Class diagram.

  • Key: ROIS11-29
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    The specification would benefit in the future from an overview Package and UML Class diagram [class names only].

    Two extra layers of Packages named “RoIS” around the Packages containing real model elements. While strictly speaking not wrong, those extra onion layers look a bit odd.

  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 07:08 GMT
  • Disposition: Deferred — RoIS 1.1
  • Disposition Summary:

    This Issue was not resolved in this Task Force and was automatically deferred to the next Task Force

  • Updated: Tue, 12 Jul 2016 14:53 GMT

"Overview" section is a bit misplaced.

  • Key: ROIS11-28
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    The specification starts with an “Overview” section, that has useful information to understand the specification. However it is placed before Clause 1 “Scope” and therefore a bit misplaced, from a standards document layout point of view. It would be good if a follow-on RTF would place this material into a more appropriate location, preferably as a new, informative Clause 7 and move the existing Clauses 7 and higher up by one number.

    The new Clause 7 should then be combined with the material from the existing Clause 7.2, which has good explanations, but is a bit misplaced in the middle of normative message and interface specifications.

  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 07:03 GMT
  • Disposition: Deferred — RoIS 1.1
  • Disposition Summary:

    This Issue was not resolved in this Task Force and was automatically deferred to the next Task Force

  • Updated: Tue, 12 Jul 2016 14:53 GMT

Atomic allocation of multiple HRI Components required

  • Key: ROIS11-3
  • Legacy Issue Number: 17133
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    Methods "search," "bind" and "bind_any" may be extended for allocating
    multiple components at once avoiding race condition.

  • Reported: RoIS 1.0b1 — Mon, 20 Feb 2012 05:00 GMT
  • Disposition: Deferred — RoIS 1.1
  • Disposition Summary:

    This issue should be deferred to the future revision.

    This issue is not discussed thoroughly and there are several opinions to resolve this so I propose to defer the issue to the future revision.

  • Updated: Tue, 12 Jul 2016 14:53 GMT

Ambiguous definition of "Condition"

  • Key: ROIS11-2
  • Legacy Issue Number: 17138
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    "Condition" is used in many interfaces and method, but it is difficult
    to implement without example.

  • Reported: RoIS 1.0b1 — Mon, 20 Feb 2012 05:00 GMT
  • Disposition: Deferred — RoIS 1.1
  • Disposition Summary:

    This issue should be deferred to the future revision.

    This issue is not discussed thoroughly and there are several opinions to resolve this so I propose to defer the issue to the future revision.

  • Updated: Tue, 12 Jul 2016 14:53 GMT

Dependency among HRI Components should be defined

  • Key: ROIS11-1
  • Legacy Issue Number: 17134
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    Dependencies (Requisite conditions) among HRI Components should be
    defined in HRI Profile.

  • Reported: RoIS 1.0b1 — Mon, 20 Feb 2012 05:00 GMT
  • Disposition: Deferred — RoIS 1.1
  • Disposition Summary:

    This issue should be deferred to the future revision.

    This issue is not discussed thoroughly and there are several opinions to resolve this. Though a resolution is partially described in a new example of Wheel Chair Robot in C.4 but
    I propose to defer the issue to the future revision.

  • Updated: Tue, 12 Jul 2016 14:53 GMT

CORBA IDL needs to be corrected to be conformant with CORBA standard

  • Key: ROIS11-23
  • Status: closed  
  • Source: AIST ( Dr. Toshio Hori)
  • Summary:

    There are several errors in CORBA IDL in Section 7.7.2 which should be corrected.

    1. missing typedefs such as DateTime and Integer.
    2. several TYPOS such as extra semicolons and commas in typedefs and parameter list.

    These should be corrected to conform to CORBA standard.

  • Reported: RoIS 1.0 — Thu, 18 Jun 2015 05:00 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Correct errors in CORBA IDL files

    CORBA PSM (CORBA IDL files) in Section 7.7.2 is corrected as follows:

    1. missing typedefs are added in RoIS_HRI.idl (DateTime and Integer).
    2. There are TYPOs such as extra commas and semicolons, and curly braces are misused which should be parentheses.
  • Updated: Tue, 29 Mar 2016 15:10 GMT

C++ headers need to be corrected

  • Key: ROIS11-22
  • Status: closed  
  • Source: AIST ( Dr. Toshio Hori)
  • Summary:

    There are errors in C++ header files which should be corrected.

    1. missing typedefs such as DateTime and Integer.
    2. 'vector' is used without specifying its namespace, which should be 'std::vector'
    3. a header file, RoIS_Person Identification.h, is included in the document twice.
    4. several TYPOs such as extra commas at the end of parameters and misused curly braces which should be parentheses.

    These should be corrected so that the header files can be compiled.

  • Reported: RoIS 1.0 — Thu, 18 Jun 2015 04:53 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Correct errors in C++ headers

    C++ PSM (C++ headers) in Section 7.7.1 is corrected as follows:

    1. missing typedefs are added in RoIS_HRI.h (DateTime and Integer) and RoIS_Service.h (DateTime).
    2. all the occurrences of 'vector' are modified as 'std::vector' to conform to C++ standard namespace.
    3. RoIS_Person_Identification.h is included twice in the document so remove one.
    4. There are TYPOs such as extra commas and semicolons at the end of parameter list, and curly braces are misused which should be parentheses.
  • Updated: Tue, 29 Mar 2016 15:10 GMT

"operable time" in System Information should be modified to deal with multiple time spans

  • Key: ROIS11-4
  • Status: closed  
  • Source: AIST ( Dr. Toshio Hori)
  • Summary:

    A field "operable time" in System Information is defined as a single 'Date Time' value in ver.1.0, but service time (i.e. operable time) of a robot system may be described as multiple time spans, such as "9AM-5PM, Mon-Fri and 10AM-5PM, Sat&Sun".
    Single 'Date Time' data type cannot deal with such use case so the data type of this field should be modified.

  • Reported: RoIS 1.0 — Wed, 15 Apr 2015 04:17 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Parameter type of 'operable time' is modified to be a list of DateTime.

    in Type of operable_type is changed to List<DateTime[ISO8601]>.

    Reference to non-normative document [W3C-DT] is changed to [ISO8601] as the normative one.

    1. Description of result 'operable time' in engine_status method in Table 7.37 in Chapter 7.6.1 is modified as follows.
    2. Figure 17 is updated to conform to the description.
  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

Change cited version of RLS from 1.0 to 1.1

  • Key: ROIS11-21
  • Status: closed  
  • Source: AIST ( Dr. Toshio Hori)
  • Summary:

    Robotic Localization Service (RLS) was revised after RoIS 1.0 was submitted so its reference is changed from 1.0 to 1.1.
    Reference URLs in C++ header files are also changed accordingly.

  • Reported: RoIS 1.0 — Thu, 18 Jun 2015 04:44 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Reference to RLS including C++ header files are updated from 1.0 to 1.1

    To follow the revision of Robotic Localization Service (RLS), normative reference and reference URLs in C++ header files are updated.

    1. In Section 3.1 (Normative References), version number is changed to 1.1 and its year is changed to 2012.
    2. In Section 7.7.1 (C++ PSM), all the occurrences of reference URLs are changed to point to 'http://www.omg.org/spec/RLS/20110501/Architecture.hpp'
  • Updated: Tue, 29 Mar 2016 15:10 GMT

Definition of timestamp refers to non normative document [W3C-DT].

  • Key: ROIS11-6
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    Definition of timestamp refers to non normative document [W3C-DT] in section 7.6.
    It should refer to [ISO8601].

  • Reported: RoIS 1.0 — Fri, 15 May 2015 08:43 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    *Type 'DateTime [W3C-DT]' changed to 'DateTime[ISO8601]'. *

    Definition of time stamp refers to non normative document [W3C-DT] in section 7.4.2.2, 7.4.3.4, 7.4.3.6 and through out the section 7.6. It should refer to [ISO8601].

    This change affects: Table 7.6 (in section 7.4.2.2), Table 7.21 (in section 7.4.3.4), Table 7.23 (in section 7.4.3.6) and Table 7.37 through 7.46 (in section 7.6).

  • Updated: Tue, 29 Mar 2016 15:10 GMT

CORBA PSM fails to compile. It should implement data types imported from RLS 1.1.

  • Key: ROIS11-40
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    CORBA PSM fails to compile. It lacks proper typedef to imported types. It does not include RLS (and ISO19XXX) definitions as well as C++ PSM.

  • Reported: RoIS 1.0 — Mon, 7 Dec 2015 23:45 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    CORBA PSM fails to compile. It should implement data types imported from RLS 1.1.

    Just as same as C++ PSM (issue ROIS11-39), CORBA PSM does not contain definitions of RLS (and ISO19XXX) related elements. The IDL files also lacks proper typedef to imported types.
    As a solution RLS related element is implemented in RoIS_HRI.idl. Other IDL files modified to include RoIS_Common.idl (and indirectly RoIS_HRI.idl) to import corresponding types and then typedef definitions are added in each module.

    RLS related definitions are included in five files: RoIS_Face_Localization.idl, RoIS_Navigation.idl, RoIS_Person_Localization.idl, RoIS_Sound_Localization.idl, RoIS_System_Information.idl.
    Typedef of 'RoLo_DataSeq' is added to each module to define corresponding variables.
    In addition, as a trivial revision, unnecessary 'public' label is removed from RoIS_Person_Identification.idl.

  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

C++ PSM should implement data types imported from RLS 1.1.

  • Key: ROIS11-39
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    Even though RLS machine readable files are imported, RoIS C++ PSM is not able to compile because it does not contain ISO19XXX definitions.

  • Reported: RoIS 1.0 — Mon, 7 Dec 2015 23:37 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    C++ PSM should implement data types imported from RLS 1.1.

    There are no definition of RLS 1.1 related elements in RoIS 1.1 C++ PSM. The machine consumable files are therefore incomplete to be compiled. Even though RLS machine readable files are imported, they does not implement ISO19XXX definitions.
    As a possible and concise solution, the RLS-related definition(s) are implemented in RoIS_HRI.hpp, which is included by ALL other header files.

    In addition to this issue, editorial fixes to RoIS11-31 are also included in the attachment file. #ifndef, #define directives were capitalized so modified them into lower-case.

  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

Add an organization to Acknowledgment.

  • Key: ROIS11-41
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    Since the contribution from ATR was done under a research project of Ministry of Internal Affairs and Communication, Japan, both ATR and MIC should be listed in the supported by part of Acknowledgment.

  • Reported: RoIS 1.0 — Tue, 8 Dec 2015 00:01 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Add an organization to Acknowledgment.

    Since the contribution from ATR was done under a research project of Ministry of Internal Affairs and Communications, Japan, both ATR and MIC should be listed in the supported by part of Acknowledgment.

  • Updated: Tue, 29 Mar 2016 15:10 GMT

Component Profiles are not valid against the XML Schema dtc/15-06-11.xsd.

  • Key: ROIS11-32
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    When validating FaceDetection.xml against the XML Schema dtc/15-06-11.xsd, a list of errors are reported.

    • Attribute 'id' must appear on element 'rois:HRIComponentProfile'.
    • Attribute 'codeSpace' must appear on element 'gml: identifier'.
    • 'urn:x-rois:def:Component:OMG::RoISCommon' is not a valid name for 'NCName'.
    • The value 'urn:x-rois:def:Component:OMG::RoISCommon' of element 'rois:SubComponentProifle' is not valid.
    • Cannot resolve 'rois:EventDetailMessageProfileType' to a type definition for element 'rois:MessageProfile'.
  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 07:32 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Component Profiles are not valid against the XML Schema dtc/15-06-11.xsd.

    When validating FaceDetection.xml against the XML Schema dtc/15-06-11.xsd, a list of errors are reported.

    • Attribute 'id' must appear on element 'rois:HRIComponentProfile'.
    • Attribute 'codeSpace' must appear on element 'gml: identifier'.
    • 'urn:x-rois:def:Component:OMG::RoISCommon' is not a valid name for 'NCName'.
    • The value 'urn:x-rois:def:Component:OMG::RoISCommon' of element 'rois:SubComponentProifle' is not valid.
    • Cannot resolve 'rois:EventDetailMessageProfileType' to a type definition for element 'rois:MessageProfile'.

    The section 7.7.3 XML PSM is newlly added in this RTF according to the issue RoIS11-19. Since this issue is also applied to the newly added section, the modifications do not appear in the specification changebar file.

  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

C++/CORBA PSMs lack Include protection.

  • Key: ROIS11-31
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    The set of C++ header files included lack proper include protection. [From a procedural point of view this is considered as an editorial correction since their technical content is identical with the files resulting from the revision process.]

    Also, each C++ header specifies its own namespace, which reflects the name of the header. While this is not wrong, it would be advisable to make those namespaces more unique, for example with a “rois” prefix, or enclose them within an outer rois namespace. [this is a suggestion for the future and not an error in the current revision]

  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 07:17 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    C++/CORBA PSMs lack Include protection.

    The set of C++ header files included lack proper include protection. From a procedural point of view this is considered as an editorial correction since their technical content is identical with the files resulting from the revision process.

  • Updated: Tue, 29 Mar 2016 15:10 GMT

Empty comments in XML files should be removed.

  • Key: ROIS11-30
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    In RoISCommon.xml (and other xml files) are empty comments for "Event Message", "Error Message" and "Parameter". [This is not per se wrong but cosmetically not so pleasing. If you choose to remove them, that would be editorial]

  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 07:11 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Empty comments in XML files should be removed.

    In RoISCommon.xml (and other xml files) are empty comments for "Event Message", "Error Message" and "Parameter". This is not per se wrong but cosmetically not so pleasing. Removal of them would be editorial.

    The section 7.7.3 XML PSM is newlly added in this RTF according to the issue RoIS11-19. Since this issue is also applied to the newly added section, the modifications do not appear in the specification changebar file.

  • Updated: Tue, 29 Mar 2016 15:10 GMT

Reference to UML specification should be updated from 2.3 to 2.4.1.

  • Key: ROIS11-27
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    RoIS 1.0 refers to UML 2.3 as a normative reference. It should refer to UML 2.4.1 (dated version 20110701).

  • Reported: RoIS 1.0 — Sat, 7 Nov 2015 06:53 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Reference to UML specification should be updated from 2.3 to 2.4.1.

    RoIS 1.0 refers to UML 2.3 as a normative reference. It should refer to UML 2.4.1 (dated version 20110701).

  • Updated: Tue, 29 Mar 2016 15:10 GMT

XML Profiles of Basic Component is missing.

  • Key: ROIS11-19
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    XML Profiles of Basic Component is missing. To be added as a part of PSM as well as C++/CORBA PSM.

  • Reported: RoIS 1.0 — Sun, 17 May 2015 00:37 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Add a section of XML Component Profile as a PSM.

    In RoIS 1.0, section 7.7.3 describes XML schema definition of HRI Component Profile but examples of Component Profiles of Basic HRI Component in XML is missing. (partly shown in Appendix A.)

    Here I propose to insert a new section of 'XML PSM' as section 7.7.3 with attached XML profiles, as well as C++/CORBA PSM in the previous sections.

    Rename the current 7.7.3 as '7.7.4 XML PSM of Component Profile Schema.'

  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

Proposal to add a new example of WheelChairComponent as C.4.

  • Key: ROIS11-16
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    As a new example of User-defined HRI Component, WheelChairComponent was proposed in 2014 Boston meeting and it was suitable to be added as section C.4.

  • Reported: RoIS 1.0 — Sat, 16 May 2015 22:59 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    A new example of WheelChairComponent was added as section C.4.

    A new example of WheelChairRobot component that extends existing Navigation component was proposed as Section C.4.

  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments:

DateTime in User-defined HRI Component should refer to [ISO8601].

  • Key: ROIS11-13
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    As well as the fix of DateTime in Section 7.6, DateTime should refer to [ISO8601] instead of [W3C-DT] through Section C.

  • Reported: RoIS 1.0 — Sat, 16 May 2015 22:13 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    DateTime modified through Section C.

    Definitions of time stamp refers to non normative document [W3C-DT] are modified to refer to [ISO8601].

    In Table C.1, definitions of parameter timestamp, timestamp_speech_start and timestamp_speech_end were modified.
    Other timestamps in Table C.4 and Table C.5 were also modified.

  • Updated: Tue, 29 Mar 2016 15:10 GMT

inconsistent use of vector and List in C++ PSM.

  • Key: ROIS11-11
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    In the definition of Move component in C++ PSM, List is used instead of vector.
    Other part in the C++ PSM consistently use vector and there seems no reason to use List.

  • Reported: RoIS 1.0 — Sat, 16 May 2015 16:44 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Use of List in C++ PSM is modified to vector.

    In the C++ PSM, use vector consistently instead of List.
    Use of List in the definition of Move Component (Move.h) are replaced with vector.

  • Updated: Tue, 29 Mar 2016 15:10 GMT

Description of a parameter is missing.

  • Key: ROIS11-9
  • Status: closed  
  • Source: JARA ( Mr. Koji Kamei)
  • Summary:

    In the definition of Person Localization component, get_parameter method should return two parameters such as detection threshold and detection interval.
    The former is duplicated and the latter is missing.

  • Reported: RoIS 1.0 — Sat, 16 May 2015 16:03 GMT
  • Disposition: Resolved — RoIS 1.1
  • Disposition Summary:

    Missing parameter 'minimum interval.'

    As result values of get_parameter method of Person Localization component, definition of 'minimum interval' parameter was missing but 'detection threshold' was duplicated.
    A parameter 'minimum interval' should be defined here.

    1. Description of result value 'minimum interval' in get_parameter method in Table 7.39 in Chapter 7.6.3 is modified as follows.
    2. Figure 19 is updated to conform to the description.
  • Updated: Tue, 29 Mar 2016 15:10 GMT
  • Attachments: