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

enable non-data columns in aggregated boiler_fuel_eia923 table #1847

Open
cmgosnell opened this issue Aug 17, 2022 · 0 comments
Open

enable non-data columns in aggregated boiler_fuel_eia923 table #1847

cmgosnell opened this issue Aug 17, 2022 · 0 comments
Labels
eia923 Anything having to do with EIA Form 923

Comments

@cmgosnell
Copy link
Member

Right now in pudl.transform.eia923.boiler_fuel() we are dropping a lot of columns (which we want to fix because of #509 more broadly and also is necessary for the deeper fuel allocation revamp #1608 (comment). Now that we are grabbing all of the input columns on extraction (via #1836) we have the ability to keep the non-data boiler-fuel columns that are either important for analysis (re reporting_frequency_code) or useful for harvesting.

BUT this table goes through an aggregation step that right now restricts the columns to only PK or data columns in _aggregate_duplicate_boiler_fuel_keys. This was uncovered via #1836 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eia923 Anything having to do with EIA Form 923
Projects
None yet
Development

No branches or pull requests

1 participant