Fixes #3005 and #3592: Hide null ORCID fields in contributor representation and correct flipped starting/ending balance for FERC1 liabilities

migrations/versions/a56fa0739cd7_add_my_cool_table.py

@@ -0,0 +1,28 @@
+"""Add my cool table
+
+Revision ID: a56fa0739cd7
+Revises: 450d100cd30b
+Create Date: 2024-11-16 20:46:01.862062
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = 'a56fa0739cd7'
+down_revision = '450d100cd30b'
+branch_labels = None
+depends_on = None
+
+
+def upgrade() -> None:
+    # ### commands auto generated by Alembic - please adjust! ###
+    pass
+    # ### end Alembic commands ###
+
+
+def downgrade() -> None:
+    # ### commands auto generated by Alembic - please adjust! ###
+    pass
+    # ### end Alembic commands ###

src/pudl/metadata/classes.py

@@ -897,11 +897,19 @@ class Contributor(PudlMeta):
         "work package leader",
     ] = "project member"
     organization: String | None = None
-    orcid: String | None = None
+    orcid: String | None | None = None
+
+    def __repr__(self):
+        """Representation excluding None values."""
+        attrs = {k: v for k, v in self.__dict__.items() if v is not None}
+        attr_str = ", ".join(f"{k}={repr(v)}" for k, v in attrs.items())
+        return f"{self.__class__.__name__}({attr_str})"
 
     @staticmethod
     def dict_from_id(x: str) -> dict:
         """Construct dictionary from PUDL identifier."""
+        # data = copy.deepcopy(CONTRIBUTORS[x])
+
         return copy.deepcopy(CONTRIBUTORS[x])
 
     @classmethod

src/pudl/transform/params/ferc1.py

@@ -4649,8 +4649,8 @@
                     "row_seq": "row_seq",
                     "row_prvlg": "row_prvlg",
                     "report_prd": "report_prd",
-                    "beg_yr_bal": "ending_balance",
-                    "end_yr_bal": "starting_balance",
+                    "beg_yr_bal": "starting_balance",
+                    "end_yr_bal": "ending_balance",
                     "dr_acct_num": "account_detail",
                     "dr_amount": "decrease_in_other_regulatory_liabilities",
                     "credits": "increase_in_other_regulatory_liabilities",