open-contracting-extensions / public-private-partnerships

OCDS for Public Private Partnerships (PPPs)
http://standard.open-contracting.org/profiles/ppp/latest/en/
Other
6 stars 3 forks source link

Which codelists to include on codelists.md? #233

Closed jpmckinney closed 3 years ago

jpmckinney commented 3 years ago

The codelists listed are:

From the extensions used, this excludes:

We can either:

  1. Include all codelists from all extensions used
  2. Include only codelists used by framework.md

If the former, the solution is easy (add the 5 from the second list). If the latter, we need to review which of the codelists are actually used by the profile. For example, we don't use bidStatistics.csv anywhere.

In fact, why is the bid extension in the profile at all? It's used in the example, but that section of the example is never referenced. (This might lead to a review of which extensions to include in the first place.)

jpmckinney commented 3 years ago

Once this issue is closed, I can update #234 and then ask CDS to translate the iterative improvements.

jpmckinney commented 3 years ago

If we review which extensions to include, we can check each off as we validate which ought to be included.

Used in examples/full.json, but otherwise not referenced?

To be removed:

Explicitly referenced in guidance:

Explicitly referenced in :jsonpointer::

And lastly:

jpmckinney commented 3 years ago

My preference is to omit extensions that aren't mentioned in the guidance, from both the profile's schema and from the example.

There are many extensions that are relevant to PPPs that aren't in the profile (given that many/most extensions have now been authored after OCDS for PPPs was), but they aren't included here. Publishers are expected to consider such extensions in the normal course of publishing OCDS.

jpmckinney commented 3 years ago

Going back to the issue description, if we omit the unchecked extensions, then I'm fine with just including all codelists from all extensions in the profile, since we'll have ended up with the same number of codelists that we have now (after removing 2 bids and 3 requirements codelists).

duncandewhurst commented 3 years ago

Once https://github.com/open-contracting-extensions/public-private-partnerships/pull/245 is merged, bids/details will appear in the framework reference.

The example is based on Red Compartida's implementation of the World Bank framework, which is why it includes some extensions and fields that are not explicitly referenced in the framework reference. So that learning from that implementation is not lost, we should consider whether to update the framework reference to mention these extensions and what they can be used to publish.

jpmckinney commented 3 years ago

Are the extensions directly related to content from The World Bank Framework for Disclosure in Public Private Partnership Projects, or were they just useful for disclosing PPPs in the case of Red Compartida?

If the latter, then they don't belong as part of the consolidated extension, and should be opt-in. If so, we can create a separate issue to add additional guidance pages for using extensions relevant to PPPs.

duncandewhurst commented 3 years ago

bids

See above, this should be included in the consolidated extension.

documentation_details

Although the fields in this extension are not explicitly referenced in the Markdown content on the framework reference page, prior to https://github.com/open-contracting-extensions/public-private-partnerships/commit/daf991e0479b95dac7a3d94baa80954e9892419c the fields in this extension were displayed in the schema tables shown on that page.

The concepts in the extension are not explicitly referenced in the WB framework, but the extension was developed to meet needs identified during the initial development of OCDS for PPPs, including:

As such, I think documentation_details should still be included in the consolidated extension. We should also restore at least one Document schema table to the framework reference so that implementers are aware of the extra fields and can consider collecting and publishing them.

process_title

I'm not sure what purpose this is serving. In the example, the same values are provided in title and project/title and description and project/description.

The framework reference only mentions the fields under project so we should remove process_title from the consolidated extension and remove its fields from the example.

budget, requirements, transaction_milestones

These extensions were useful for Red Compartida, but I'm happy for them to be opt-in otherwise.

jpmckinney commented 3 years ago

Sounds good to keep bids and documentation_details, and to remove the rest.

I think all "Document" links point to https://standard.open-contracting.org/profiles/ppp/latest/en/reference/documents/, which has a table with all extended fields.