-
-
Notifications
You must be signed in to change notification settings - Fork 106
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 straggler dbf-only factoids into ferc1 table metadata #2716
add straggler dbf-only factoids into ferc1 table metadata #2716
Conversation
src/pudl/transform/ferc1.py
Outdated
"ferc_account": [pd.NA], | ||
"xbrl_factoid_original": ["accumulated_deferred_income_taxes"], | ||
"intra_table_calc_flag": [True], | ||
"row_type_xbrl": ["report_value"], |
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.
This should be "reported_value" in all of the fixes.
src/pudl/transform/ferc1.py
Outdated
{ | ||
"calc_component_to_replace": {}, | ||
"calc_component_new": { | ||
"name": "unappropriated_undistributed_subsidiary_earnings_previous_year", | ||
"weight": 1.0, | ||
"source_tables": ["retained_earnings_ferc1"], | ||
}, | ||
}, |
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.
Are these changes to the calculation fixes related to the dbf-only facts?
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.
ah no this is just one of the many instances of factoids that weren't actually in the data.
src/pudl/transform/ferc1.py
Outdated
"intra_table_calc_flag": True, | ||
"row_type_xbrl": "report_value", | ||
} | ||
for dbf_onl_fact in ["special_funds_all", "nuclear_fuel"] |
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.
The computer doesn't care but did you mean for this to be dbf_only_fact
?
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.
lol yes
PR Overview
I started with just adding the dfb-only metadata fields into each table's metada doing it in a bespokey way. will probably parameterize this adding of these guys.
PR Checklist
dev
).