Skip to content
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

tx1-2: notice type handling #238

Closed
costezki opened this issue Nov 23, 2022 · 9 comments
Closed

tx1-2: notice type handling #238

costezki opened this issue Nov 23, 2022 · 9 comments
Assignees
Labels
type: bug Something implemented incorrectly in a release.
Milestone

Comments

@costezki
Copy link
Collaborator

[N Muric]

In notices covering Contract Award Notices and other types of notices, the mappings are made in each case to the Result Notice which creates a wrong mapping to the other types of notices

@costezki costezki added the type: bug Something implemented incorrectly in a release. label Nov 23, 2022
@costezki costezki added this to the 2022-12 milestone Nov 23, 2022
@costezki
Copy link
Collaborator Author

costezki commented Nov 23, 2022

In F21, F22, F23 Forms, we will use the type attribute in element when present.

The following correspondence rules will be used

award_contarct -> epo:ResultNotice
contract -> epo:CompetitionNotice
concession_award_contract -> epo:ResultNotice
pri_call_competition -> epo:CompetitionNotice
pri -> epo:CompetitionNotice

Note: "qualification system" and "periodic indicative notice" shall be considered as well as they are needed in F22

image

image

image

F13, which does not have a notice type, will be mapped to epo:ResultNotice.

@csnyulas
Copy link
Contributor

AFAICT, Qualification systems can be recognized from the presence of one of the following XPaths:
LEFTI/QUALIFICATION,
OBJECT_CONTRACT/OBJECT_DESCR/QS/,
OBJECT_CONTRACT/OBJECT_DESCR/RENEWAL, and
OBJECT_CONTRACT/OBJECT_DESCR/RENEWAL_DESC

Is checking for this XPaths is sufficient to identify if a notice is regarding a Qualification System?
Can it happen that all these fields are missing and still be a notice about a QS?
Is there a better, more secure, way to identify that a notice is a QS, based on the information available in the XML, than checking for these XPaths? The TYPE attribute of the NOTICE element does not seem to give us any indication about this.

Is there a specific class, or a single (dedicated) property in EPO that can be used to identify a QS, or it is checked through the presence of any of these properties: epo:hasQualificationSystemDuration, epo:hasQualificationSystemPeriod, epo:hasQualificationSystemRenewalDescription ?

@csnyulas
Copy link
Contributor

csnyulas commented Nov 30, 2022

Note that the epo:Notice instances created from forms that don't have the "NOTICE TYPE" attribute, should also get a second type (a subclass of epo:Notice), based on the main nature of the form. For example, for F03, F06, F25 and F13 it will be epo:ResultNotice, while for form F20 it will be epo:ContractModificationNotice.

@csnyulas
Copy link
Contributor

csnyulas commented Dec 1, 2022

Other Notice type values to be mapped (all collected from our sample data):

Form F01:
<NOTICE TYPE="PRI_CALL_COMPETITION"/>
<NOTICE TYPE="PRI_ONLY"/>
<NOTICE TYPE="PRI_REDUCING_TIME_LIMITS"/>

Form F04:
<NOTICE TYPE="PER_CALL_COMPETITION"/>
<NOTICE TYPE="PER_ONLY"/>
<NOTICE TYPE="PER_REDUCING_TIME_LIMITS"/>

Form F07:
<NOTICE TYPE="QSU_CALL_COMPETITION"/>

Form F21:
<NOTICE TYPE="AWARD_CONTRACT"/>
<NOTICE TYPE="CONTRACT"/>
<NOTICE TYPE="PRI_CALL_COMPETITION"/>

Form F22:
<NOTICE TYPE="AWARD_CONTRACT"/>
<NOTICE TYPE="CONTRACT"/>

Form F23:
<NOTICE TYPE="CONCESSION_AWARD_CONTRACT"/> 
<NOTICE TYPE="PRI"/>

@csnyulas
Copy link
Contributor

csnyulas commented Dec 7, 2022

In addition to the above, in other samples we also found:
<NOTICE TYPE="QSU_ONLY"/>

This also answers the question for how to identify "Qualification systems" and "Call for competition for Qualification systems" (with notice type QSU_ONLY and QSU_CALL_COMPETITION). These are the values that we should map for F22, for example, despite the fact that it does not appear in our test data.

BTW, the fourth possible notice type for F21 is "PRI_ONLY" despite not appearing in out test data.

This ticket can be marked fixed for the purpose of addressing the TX1.2 feedback, but before we can close it, we should also map the other notice value types, namely:

  • PRI_ONLY
  • QSU_ONLY
  • QSU_CALL_COMPETITION
    and preferably also the following ones, needed in future forms:
  • PRI_REDUCING_TIME_LIMITS
  • PER_ONLY
  • PER_CALL_COMPETITION
  • PER_REDUCING_TIME_LIMITS

@csnyulas csnyulas added the act: for closing it can be closed but an additional confirmation is needed label Dec 7, 2022
@costezki
Copy link
Collaborator Author

costezki commented Dec 9, 2022

PRI_ONLY  -> epo:PlanningNotice
QSU_ONLY -> epo:CompetitionNotice
QSU_CALL_COMPETITION -> epo:CompetitionNotice
PRI_REDUCING_TIME_LIMITS -> epo:PlanningNotice
PER_ONLY -> epo:PlanningNotice
PER_CALL_COMPETITION -> epo:CompetitionNotice
PER_REDUCING_TIME_LIMITS -> epo:PlanningNotice
if(../NOTICE/@TYPE='AWARD_CONTRACT' or ../NOTICE/@TYPE='CONCESSION_AWARD_CONTRACT') then 'http://data.europa.eu/a4g/ontology#ResultNotice' else 
if (../NOTICE/@TYPE='CONTRACT' or ../NOTICE/@TYPE='PRI' or ../NOTICE/@TYPE='PRI_CALL_COMPETITION') then 'http://data.europa.eu/a4g/ontology#CompetitionNotice' 
else null ";

csnyulas added a commit that referenced this issue Dec 9, 2022
@csnyulas
Copy link
Contributor

csnyulas commented Dec 9, 2022

Closing this, as it has been fully fixed. Opened an issue, #268, about implementing a potential improvement (related to performance an manageability) on the current solution.

@muricna
Copy link

muricna commented Jan 17, 2023

In competition Notices according to the ontology the CompetitionNotice announcesRole and the ResultNotice refersToRole or in other cases announcesRole this is not reflected in the mapping. The same can be said for announcesProcedure and refersToProcedure.
The ontology should be informed that the Notice refersToProcedure predicate should be from ResultNotice
This comment applies to all of section I and it is up to the contractor to ensure the other sections are correct for other forms and for other classes such as Lot

@csnyulas
Copy link
Contributor

Related to the last comment from @muricna: I created a separate issue for it (#326), because the problem described in the comment is different from what this issue was about. This issue is about attaching the appropriate Notice subclasses as a secondary rdf:type statement to the Notice instance. That has been fixed and tested, as can be seen in the linked commits. This is why we closed this ticket.
We will mark again this issue as Fixed, and further improvements/fixes will be addressed in issue #326.

@csnyulas csnyulas added act: for closing it can be closed but an additional confirmation is needed and removed To be fixed labels Jan 25, 2023
@valexande valexande modified the milestones: 2022-12, 2023-02 May 8, 2023
@csnyulas csnyulas removed the act: for closing it can be closed but an additional confirmation is needed label Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something implemented incorrectly in a release.
Projects
None yet
Development

No branches or pull requests

4 participants