Reuse further terms from ODRL
clange opened this issue · 2 comments
In a review post #504 and #543, I noticed that still a few terms are around that IDS redefines from ODRL but where IDS does not add value. These should also be stripped and replaced by plain ODRL.
Overall, let's keep this discussion in sync with https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/.
-
target
- anything in "codes" was not subject of the review done in #504 / #543 but should also be reviewed. At a first glance, the following terms are redundant:
-
idsc:USE
→odrl:use
-
idsc:COMPENSATE
→odrl:compensate
-
idsc:PAY_AMOUNT
→odrl:payAmount
-
idsc:EQ
→odrl:eq
(OK, maybe here there is a (small) added value in IDS = explicitly stating that this is a binary operator. However, I'd suggest we check this with the Usage Control experts around @hosseinzadeha, whether the information that an operator is binary is really needed. In any case I do not see operators that are not binary.)
-
This will be helpful for the further alignment with Gaia-X, where plain ODRL shall be used as much as possible, and thus in scope of the FDS T72 project.
For the Gaia-X perspective, note https://gitlab.com/gaia-x/technical-committee/federation-services/data-exchange/-/merge_requests/19.
-
Furthermore,
evaluation_external/README.md
says thatrightOperand
is "pending". Is the "why" documented somewhere? If not, could you please document it? -
Additionally, why is
ids:actionRefinement
needed in addition toodrl:refinement
, and why is it not a subproperty ofodrl:refinement
?actionRefinement
seems to be covered by the domain ofodrl:refinement
includingodrl:Action
;participantRefinement
might be covered by the domain ofodrl:refinement
includingPartyCollection
, which is a subclass ofParty
, which could be (or is already?) aligned withids:Participant
. -
idsc:WRITE
might be obsolete, as ODRL deprecated it in favour ofodrl:modify
. -
Could it actually be that
ids:contractOffer
is redundant and can be replaced byodrl:policy
?
Hi!
It is nice to see the usage of plain ODRL for alignment between data spaces initiatives.
A few comments:
- Regarding the operators, I understand you are defining
idsc:EQ
in the Information Model instead of usingodrl:eq
as you want to make explicit that it is a binary operator. I'm not sure if this is necessary as I think ODRL operators should always be binary. However, as I'm an active member of ODRL's CG and am involved on the ODRL's formal semantics specification (which should be the first step to hopefully have an ODRL 3.0), might I make a suggestion that you bring this issue to the CG for discussion? I would be very happy to discuss this in one of our meetings as it is something that can be used to improve at least the description of the operator concept. - Same thing for other ODRL concepts you feel need a tweak in their description -- if they are not data spaces-specific then now is a good time to suggest changes to the ODRL CG.
- Unless you have a good reason to have different types of refinements, I would stick to the usage of
odrl:refinement
. - Instead of
ids:contractOffer
, you could stick with plain ODRL and model the policy as anodrl:Offer
and useodrl:contractingParty
andodrl:contractedParty
to express the party who is offering the contract and the party who is being contracted, respectively. odrl:hasPolicy
can be used to connect resources with policies.
Happy to discuss any other alignments :)
I proceeded to make the following changes:
idsc:USE → odrl:use
(commit: 2567bde): aligned with the proposal in https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/12- i
dsc:WRITE→ odrl:modify
(commit: cfde313): aligned with the proposal in https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/16 idsc:COMPENSATE → odrl:compensate
(commit: 441e91d): aligned with the proposal in https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/15idsc:PAY_AMOUNT → odrl:payAmount
(commit: c7b660d): aligned with the proposal in https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/28idsc:PAYMENT
, added a note as suggested in https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/28 (commit: 40222f6)idsc:TARGET_POLICY
removed this property, as it is suggested here https://github.com/International-Data-Spaces-Association/ODRL-profile-for-IDS/issues/23 to directly use odrl:hasPolicy as the target policy (commit: cc4c197)idsc:EQ → odrl:eq
(commit: ab882a3) After evaluating the definition of the idsc:EQ, and considering the comment from @besteves4, I removed the idsc:EQ and replaced it with odrl:eqids:rightOperand
was replaced byodrl:rightOperand
by commit 1837136. Moreover, changes in the documentation and README file corresponding to the evaluation (commit:77b91ce8)ids:actionRefinement
andids:participantRefinement
are removed and replaced byodrl:refinement
(commit: 9adc4f4)- The change regarding
ids:contractOffer
will be consider in #608
Note: All the changes are also reflected in our document here: https://docs.google.com/spreadsheets/d/17RjLl6wkL_V2_jbf_vBfJ_77NgmENnf_tIfWZsNhr_Y/edit#gid=0