-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documentation - request for definition of Terms #86
Comments
> (To be further determined - how complete is our include/reference concept?)
|
A good start. Reading..
I think you should at least spell out SOA in the first use.
There seems to be no corresponding reference. Is it supposed to be referring to the sentence prefixed with '>' a couple of paras down? |
Better to put it in a *.md file in a PR to simplify review? Good start - I think it would be good to add at least placeholders for Events and Properties, to clarify difference between Methods/Events/Properties. |
Concerning streaming replies we had some discussion on slack in April 4th-5th in rpc channel. There I created a proposal:
With this definition a "typical" VSS read or write would be an atomic call, while a "typical" VSS subscribe would be a stream call. Magnus included some examples in Slack. This way we can already in VSC express that e.g. MoveWindow is a stream call where you can get zero or more events showing current position of window and status of the request With the definition above we can define "event" as something that is either broadcasted by the server, or received triggered by a method. For property we could possibly describe it as a shortcut for get/set-methods, e.g. if you specify property X, that gets the same results as specifying the two methods getX and setX. |
Easier for me to say at the high level, than it be proven in the details but wouldn't there be value in staying as close as possible and ideally logically equivalent, to the approach in OpenAPI and AsyncAPI? Although possibly not in definition format if that is hard to cherry pick or that does not make sense for other reasons. Equivalence or a sub-set of it supports smoother integration. Particularly for those coming from those worlds. |
I am making a different proposal that could be summarized as:
The core reason for this this is that the Methods should be defined the same way whether they are later executed as blocking or non-blocking, instead of making this choice already when the method is defined. In other words, we can delay the decision of whether a method is of the blocking or non-blocking type. In some cases this is also limited by the target environment, so if you avoid making this choice immediately then the API is more reusable in many contexts. The full text proposal is coming, so I propose to give feedback there. EDIT: I think the above is the simplest way (explicitly name the Datatype of the return value), but something that could be considered is if a return value could be equal to an already defined Event. Let's see after we have agreed on the description of Event. |
See COVESA/vehicle_service_catalog#41 for a PR where you can comment if you prefer it |
Since this is a question for the definition of the language/interface-model that is now IFEX project, the issue will be transferred from Vehicle Service Catalog project to IFEX project. |
On the previous meeting I was requested to start on a draft of terms definitions, to find consensus around the most key concepts of the VSC language. First proposal below.
The text was updated successfully, but these errors were encountered: