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
Update expected row count for EIA tables post 860m quarterly update #3380
Conversation
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.
Ready to merge once validation tests pass locally.
("bf_eia923", 1_569_568, 1_569_568, 128_252), | ||
("bf_eia923", 1_607_752, 1_607_752, 128_252), |
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.
i went and quickly looked at the build logs and it looks like this one is a typo and should be 1_670_752
E ValueError: bf_eia923: found 1607752 rows, expected 1569568. Off by 2.433%, allowed margin of 0.000%
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.
In the example above the found value 1607752
is the same as the one I have written, it's just confusing to look at with the underscores!
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.
oh you are so right! sorry!
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.
see the one lil typo
See above, confirmed that row counts are correct
Forgot to update expected rows in validation tests as part of #3313 and #3367