Skip to content
Snippets Groups Projects

Updated term definitions.

Merged Raul Garcia-Castro requested to merge issue-5 into master

Closes #5 (closed).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • OK.

    By Maria Poveda on 2019-10-15T14:48:25 (imported from GitLab project)

  • Three questions on this MR:

    1. saref:EventFunction

    Can you explain why there is no change?

    1. saref:LevelControlFunction
    • Proposal was: An actuating function that allows to do level adjustments of the property of a device or a feature of interest in a certain range (e.g., 0%-100%), such as dimming the light in a room or setting the speed of an electric motor.
    • You wrote: An actuating function that allows to do level adjustments of a property in a certain range (e.g., 0%-100%), such as dimming a light in a room or setting the speed of an electric motor.
    1. saref:hasProfile
    • Proposal was: A relationship associating a profile to a certain device or feature of interest
    • You wrote: A relationship associating a profile to a certain device

    By lefranois on 2019-10-22T10:14:32 (imported from GitLab project)

  • EventFuncion is not changed in this merge request because the definition was already updated in a previous one, when it was already made more generic.

    Regarding the other two questions, right now there is no mention to features of interest in SAREF, so it didn't make sense to update the descriptions.

    Once we include features of interest, we have to update the required descriptions accordingly. But this is part of another issue.

    By garciacastro on 2019-10-22T14:42:28 (imported from GitLab project)

  • mentioned in issue #40 (closed)

    By lefranois on 2019-10-22T15:03:46 (imported from GitLab project)

  • okay, then I just add a mention to this comment in the separate issue #40 (closed) so that we don't forget

    OK to merge

    By lefranois on 2019-10-22T15:04:13 (imported from GitLab project)

  • mentioned in commit b3e2e68f

    By garciacastro on 2019-10-23T08:23:43 (imported from GitLab project)

  • merged

    By garciacastro on 2019-10-23T08:23:43 (imported from GitLab project)

Please register or sign in to reply
Loading