open-contracting / standard

Documentation of the Open Contracting Data Standard (OCDS)
http://standard.open-contracting.org/
Other
139 stars 46 forks source link

add ImmediateContainer, Quantity and SimplyUnit, clarify Unit use #1689

Closed odscjen closed 5 months ago

odscjen commented 6 months ago

closes #1343

draft until comment is agreed upon

jpmckinney commented 6 months ago

Just a couple comments on the Markdown. I'll review the schema when ready.

odscjen commented 6 months ago

Error FAILED tests/test_schema_integrity.py::test_versioned_release_schema_is_in_sync - UserWarning: /definitions/Quantity/properties/value has unrecognized type ['string', 'number', 'null']

Quantity has been lifted straight from the medicine extension, I think the test may need updating

jpmckinney commented 6 months ago

Noting that I'll correct a "agreeemnts" typo and update the profile template via this PR before merging.

jpmckinney commented 5 months ago

I considered adding 'GDSN' to itemClassificationScheme.csv (similarly, we could add a code for the immediateContainer codelist from the medicine extension), but since we discourage units of presentation for the unit field, I think it makes sense to omit it.

duncandewhurst commented 5 months ago

I reviewed the individual commits and ran the pre-commit script to update the dereferenced release schema so that I could review the HTML documentation.

The changes look good, but the first sentence of the descriptions of Item.unit and Item.immediateContainer simply restate the title of the fields. Perhaps they could be improved? e.g.

Anyway, happy for you to merge if you think the descriptions are OK as-is.

jpmckinney commented 5 months ago

Good call - I'll reword to be more informative.

jpmckinney commented 5 months ago

The innermost container in which the items are packaged

For immediate container:

ISO_11239_2012(en).PDF

immediate packaging in which a manufactured item or pharmaceutical product is contained and with which it is in direct contact EXAMPLES Ampoule, vial, prefilled syringe, bottle, blister. NOTE 1 An immediate container can be fitted with or have integrated into it an administration device and/or closure. NOTE 2 A pharmaceutical dose form can fulfil the role of an immediate container, e.g. a capsule containing a powder for inhalation; the capsule in this case is not a container. NOTE 3 An alternative, compatible definition of immediate container (“immediate packaging”) is given in Directive 92/27/EEC. NOTE 4 Adapted from ENV 12610:1997.

Directive 92/27/EEC

immediate packaging means the container or other form of packaging immediately in contact with the medicinal product

So I'll go with:

The innermost packaging with which the item is in direct contact.

jpmckinney commented 5 months ago

And I'll simplify to "The unit for pricing the item".