-
Notifications
You must be signed in to change notification settings - Fork 3
build CEQR schools 2026 #2132
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
base: main
Are you sure you want to change the base?
build CEQR schools 2026 #2132
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| name: CEQR - Schools | ||
| on: | ||
| workflow_dispatch: | ||
| inputs: | ||
| dataset: | ||
| description: "Dataset to build" | ||
| type: choice | ||
| required: true | ||
| options: | ||
| - sca_capacity_projects | ||
| - sca_e_projections_by_boro | ||
| - sca_e_projections_by_sd | ||
| - ceqr_school_buildings | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-22.04 | ||
| defaults: | ||
| run: | ||
| shell: bash | ||
| working-directory: products/ceqr/ceqr_app | ||
| container: | ||
| image: nycplanning/build-geosupport:${{ inputs.image_tag || 'latest' }} | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Load Secrets | ||
| uses: 1password/load-secrets-action@v1 | ||
| with: | ||
| export-env: true | ||
| env: | ||
| OP_SERVICE_ACCOUNT_TOKEN: ${{ secrets.OP_SERVICE_ACCOUNT_TOKEN }} | ||
| BUILD_ENGINE_SERVER: "op://Data Engineering/EDM_DATA/server_url" | ||
| AWS_S3_ENDPOINT: "op://Data Engineering/DO_keys/AWS_S3_ENDPOINT" | ||
| AWS_SECRET_ACCESS_KEY: "op://Data Engineering/DO_keys/AWS_SECRET_ACCESS_KEY" | ||
| AWS_ACCESS_KEY_ID: "op://Data Engineering/DO_keys/AWS_ACCESS_KEY_ID" | ||
|
|
||
| - name: Setup build environment | ||
| working-directory: ./ | ||
| run: ./bash/docker_container_setup.sh | ||
|
|
||
| - name: Run recipe | ||
| run: | | ||
| export RECIPE_EGNINE=$BUILD_ENGINE_SERVER/recipe | ||
| export EDM_DATA=$BUILD_ENGINE_SERVER/defaultdb | ||
| ./ceqr run recipe ${{ inputs.dataset }} |
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. pretty sure the LCGMS we get form CAPS for CEQR schools is different that what we get from the DOE website and use in the
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we give this a different id then? Feels unlikely that it would happen by accident, but we still shouldn't have two different ingest/library templates with the same dataset id.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. bumping this comment - other than that, good to go
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. planning to rename but I think I'll have to resolve some weirdness with how this is used in this build pretty sure the build uses past versions of datasets stored in the edm-recipes DB, so maybe I'll just have to re-rename this when it's imported during the build (instead of recreated tables with the new name and changing the build code)? |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| dataset: | ||
| name: doe_lcgms | ||
| acl: public-read | ||
| source: | ||
| url: | ||
| path: s3://edm-recipes/inbox/sca/{{ version }}/doe_lcgms.csv | ||
| options: | ||
| - AUTODETECT_TYPE=NO | ||
| - EMPTY_STRING_AS_NULL=YES | ||
| geometry: | ||
| SRS: null | ||
| type: NONE | ||
|
|
||
| destination: | ||
| geometry: | ||
| SRS: null | ||
| type: NONE | ||
| options: | ||
| - OVERWRITE=YES | ||
| - PRECISION=NO | ||
| fields: [] | ||
| sql: null | ||
|
|
||
| info: | ||
| description: | | ||
| Provided by DCP Capital Planning team as an excel file | ||
| with a name like "LCGMS_SchoolData". | ||
|
|
||
| This is only needed for the legacy CEQR schools dataset ceqr_school_buildings | ||
| and is different from the doe_lcgms ingest source data used in FacDB. | ||
| url: "" | ||
| dependents: [] |
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so I don't have to run a dev container to run builds that use geosupport, modeled on the in-use
ceqr_dep_monthly.ymlaction. couldn't test this withworkflow_dispatchuntill it's mergedThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, I'm totally cool with us merging empty templates to main without review, then you can populate it in your PR and actually test it.