Skip to content

Extract RUS7 Balance sheet table#4910

Merged
cmgosnell merged 1 commit intorus7-extractfrom
rus7-extract-table2
Jan 12, 2026
Merged

Extract RUS7 Balance sheet table#4910
cmgosnell merged 1 commit intorus7-extractfrom
rus7-extract-table2

Conversation

@cmgosnell
Copy link
Member

@cmgosnell cmgosnell commented Jan 9, 2026

Overview

Working on #4897.

What problem does this address?

What did you change?

Documentation

Make sure to update relevant aspects of the documentation:

  • Update the release notes: reference the PR and related issues.
  • Update relevant Data Source jinja templates (see docs/data_sources/templates).
  • Update relevant table or source description metadata (see src/metadata).
  • Review and update any other aspects of the documentation that might be affected by this PR.

Testing

How did you make sure this worked? How can a reviewer verify this?

To-do list

  • If updating analyses or data processing functions: make sure to update row count expectations in dbt tests.
  • Run pixi run pre-commit-run to run linters and static code analysis checks.
  • Run pixi run pytest-ci locally to ensure that the merge queue will accept your PR.
  • Review the PR yourself and call out any questions or issues you have.
  • For PRs that change the PUDL outputs significantly, run the full ETL locally and then run the data validations using dbt. If you can't run the ETL locally then run the build-deploy-pudl GitHub Action manually and ensure that it succeeds.

@cmgosnell cmgosnell self-assigned this Jan 9, 2026
@cmgosnell cmgosnell requested a review from aesharpe January 9, 2026 21:53
@cmgosnell cmgosnell moved this from New to In review in Catalyst Megaproject Jan 9, 2026
@cmgosnell cmgosnell added new-data Requests for integration of new data. rus7 USDA Rural Utilities Services Form 7 -- Financial and Operating Report: Electric Distribution labels Jan 9, 2026
@aesharpe
Copy link
Member

aesharpe commented Jan 12, 2026

Does it make sense / do you want to have skiprows / skipheaders / init files or is that not necessary?

@cmgosnell
Copy link
Member Author

the skiprows etc. isn't necessary for the csv extractors! (we've only ever seen csv's that just start on the first row thus far)

@cmgosnell cmgosnell merged commit 75dbb60 into rus7-extract Jan 12, 2026
13 of 14 checks passed
@cmgosnell cmgosnell deleted the rus7-extract-table2 branch January 12, 2026 17:50
@github-project-automation github-project-automation bot moved this from In review to Done in Catalyst Megaproject Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-data Requests for integration of new data. rus7 USDA Rural Utilities Services Form 7 -- Financial and Operating Report: Electric Distribution

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants