-
Notifications
You must be signed in to change notification settings - Fork 46
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
Avoid examples that are incoherent, or that are too abstract #1449
Comments
cc @sabahfromlondon to identify/prioritize the pages on which to add/improve examples. |
@jpmckinney is it ok if I start working on this or shall I leave it for Sabah? |
@jpmckinney I think I missed this! Sorry. I'll compile the list this week. Thanks for flagging @odscjen |
@jpmckinney I've completed a review of the documentation pages. I also thought it was worth checking the depper pages, even though that was not quite within the scope of this ticker. It didn't take much longer and I have some thoughts / suggestions it might be worth discussing.. Here's the link to the spreadsheet. Shall we discuss on our call on Wednesday? cc: @odscjen |
Thank you! We discussed and answered some of the questions in the sheet. I'll leave this with analysts to find appropriate examples. |
The sheet has identified 2 required tasks and 9 potential tasks Example Required
@jpmckinney for the 'Maybe' examples, is the sheet the best place to ask clarifying questions/ have discussions, or would you rather we brought it here as tasks? |
@neelima-j I think your first link is meant to be to https://standard.open-contracting.org/latest/en/primer/how/? For the maybe examples, let's move it here as notifications are easier to track. |
Maybe examples copied in from the spreadsheet. Initial comments also from the spreadsheet, italic comments added by me as I transferred this list to here:
|
@yolile Which of these would you prioritize for added/improved examples? Are any good enough as-is? |
We often hear and need to show partners what OCDS data looks like, so this sounds good. The only thing to consider when preparing the example is to make sure that we state that this is just an example and not all the example fields are required nor does the example contain all the existing fields etc.
For this one, I think we have worked examples at the Merging section and the Change History guidance. Maybe we can just add a sentence saying that an example is available on the Change History page?
The tabular examples are tabular on purpose to easily demonstrate the issue and solution. I think we can leave it as it is.
Leave it as it is as there is no special mapping to show
Agree that the example should be in JSON and not in tabular format.
The identifiers section has a lot of things to be improved and it has its own issues already documented. For all the others, do we really want/need real-world examples? I can see some value in having a real example to follow, but I'm afraid of 1) broken links in the future and 2) the reader following any bad practice that the referenced publisher might have. We could avoid both things by only adding a copy of the real JSON file but I don't see much difference with having fictional examples (other than knowing who else is publishing that in OCDS) |
@yolile When this issue was created, we had identified that users find it harder to relate to an example if it is totally fictional, e.g. "Department of Bubble Gum buys 10 blister packs of chewing gum from Happy Smiles Inc." If our fictional examples are close enough to "real", then they are fine. If there are deficiencies in a publisher's data, we can just correct them in the documentation - we aren't making a promise that the data is identical. Edit: Another reason for using real data was to avoid creating incoherent examples. We sometimes had such examples in the past, where we wanted to demonstrate a lot of fields at once, but it started to make no sense in reality. |
For the release and record of the same dataset, Ghana publish records which include versioned and compiled releases so they're a good example for this e.g use ocds-uhveoc-680235 from https://www.ghaneps.gov.gh/ocds/services/recordpackage/getrecordpackage/2023/10 which has 2 releases. However the data will need cleaned up, e.g. they're not used Ebonyi State in Nigeria could be used for the json and csv example, they provide both json and excel downloads, e.g. https://ebonyieprocure.eb.gov.ng/do_award_contract_details.php?id=ocds-zinqhl-014665-EB/WTR/EP002-NG. Again this will need corrected, e.g. I've picked African examples for both of these as: @jpmckinney @yolile if you're happy with those examples I'll get started on drafting them up taking into consideration the need to make it clear that the actual data may not be identical to the data the publisher has published. Re. the other points are you happy with:
I've updated the above comment that lists the maybes to strike out those that @yolile has already confirmed can be left as is |
@yolile I defer to you. |
All sounds good to me, thanks @odscjen |
I'm following up on @jpmckinney's request in #1661 (comment) for me to see if any of the existing examples mentioned in this issue are deficient, and if so, propose some improvement. Regarding avoiding totally fictional or incoherent examples (the purpose of this issue), I propose the following improvements:
Having reviewed the examples mentioned in this issue, I think there are many other opportunities for improvement. In most cases, with respect to the concepts they are supposed to illustrate, the examples are either unnecessarily complex or include fields which are irrelevant, or both. My other observation from reviewing the changes in #1661 is that it's more trouble than it's worth to find real OCDS data and to try to correct errors and trim it down for brevity and clarity so that it can serve as example data. I think that we should:
I'm sharing my notes from reviewing the examples mentioned in this issue so that they aren't lost:
|
All sounds good to me – thank you, @duncandewhurst ! You can create an issue about "Update the contributor guidelines" in the handbook. |
It does – we had planned to remove it as part of a full redesign, which hasn't occurred. |
Moved to #1666 (comment) |
Moved to #1666 (comment) |
I've updated the issue title to reflect the purpose expressed in #1661 (comment) and I've moved the previous two comments to #1666 (comment) (I'd forgotten I'd opened a separate issue about making examples minimal). |
Whilst reviewing examples for #1666, I noted a couple more overly generic examples, which I'll update as part of #1680:
|
@duncandewhurst I think the second checklist item is not part of #1680, correct? Is there anything else left from this issue, otherwise? |
Whoops, yes. I should've listed it in #1666. Let me know if I should add it to #1680
Nope, I think we're done. |
Sure, should be quick. |
Done in 6f058dc |
Users prefer specific examples. We now have many specific examples on deeper pages (e.g. for the Map step).
However, much of the older content still has generic/abstract examples – or no examples.
We should review the documentation to check for examples that need to be added or improved. We can then list the pages as a to-do list in this issue, and close the issue once a sufficient number of pages have been updated.
The text was updated successfully, but these errors were encountered: