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

Add field lists to CoST IDS mapping files, update core CoST mapping to reflect latest schema #483

Merged
merged 20 commits into from
Mar 28, 2024

Conversation

duncandewhurst
Copy link
Contributor

@duncandewhurst duncandewhurst commented Mar 15, 2024

Related issues

Description

  • Add a list of the OC4IDS fields used in each mapping to the files in mapping/. For the .csv files, the field lists are hand-authored. For sustainability.yaml, the field lists are generated from the examples for each mapping, by manage.py update-sustainability-fields. These field lists are used to calculate coverage of the CoST IDS and to filter the OC4IDS field-level mapping template by CoST IDS element (work in progress).
  • Replace smart quotation marks with simple quotation marks and use consistent column headers in CSV mapping files ("CoST IDS draft definition" -> "Description").
  • Update core CoST IDS mapping to reflect latest schema (budget/budgetBreakdown > budget/budgetBreakdowns/budgetBreakdown)

Merge checklist

After merging

@duncandewhurst duncandewhurst changed the title A field lists to OC4IDS mapping files Add field lists to OC4IDS mapping files Mar 27, 2024
@duncandewhurst duncandewhurst changed the title Add field lists to OC4IDS mapping files Add field lists to OC4IDS mapping files, update core CoST mapping to reflect latest schema Mar 27, 2024
@duncandewhurst duncandewhurst changed the title Add field lists to OC4IDS mapping files, update core CoST mapping to reflect latest schema Add field lists to CoST IDS mapping files, update core CoST mapping to reflect latest schema Mar 27, 2024
@duncandewhurst duncandewhurst marked this pull request as ready for review March 27, 2024 03:08
mapping/process-level-implementation.csv Outdated Show resolved Hide resolved
mapping/process-level-procurement.csv Outdated Show resolved Hide resolved
mapping/process-level-procurement.csv Outdated Show resolved Hide resolved
mapping/process-level-procurement.csv Outdated Show resolved Hide resolved
mapping/process-level-procurement.csv Outdated Show resolved Hide resolved
@duncandewhurst duncandewhurst requested review from jpmckinney and removed request for neelima-j March 28, 2024 20:14
@jpmckinney jpmckinney force-pushed the machine-readable-mapping branch from 06cde70 to 0112732 Compare March 28, 2024 21:15
@@ -1068,5 +1068,45 @@ def update_sustainability_docs():
f.writelines(docs)


@cli.command()
def update_sustainability_fields():
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@duncandewhurst duncandewhurst merged commit b95be6a into 0.9-dev Mar 28, 2024
10 checks passed
@duncandewhurst duncandewhurst deleted the machine-readable-mapping branch March 28, 2024 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants