-
-
Notifications
You must be signed in to change notification settings - Fork 108
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 PUDL to use frictionless v5 #3569
Conversation
fffc3a9
to
b79cc9b
Compare
- jsonschema=4.21.1=pyhd8ed1ab_0 | ||
- jsonschema-specifications=2023.12.1=pyhd8ed1ab_0 | ||
- jsonschema-with-format-nongpl=4.21.1=pyhd8ed1ab_0 | ||
- jsonschema=4.17.3=pyhd8ed1ab_0 | ||
- jsonschema-with-format-nongpl=4.17.3=pyhd8ed1ab_0 |
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.
Note that frictionless
is currently pinning to jsonschema<4.18
and jupyter_events
also depends on that package, which results in a downgrade of some jupyter packages below.
1e5d4a4
to
3403a5d
Compare
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 decided to just "commit suggestion" since there's no sense in going back-and-forth just to delete an obsolete TODO. Approved.
TODID TODONE |
Silly stochastic unit test failure. Re-running. |
…ove defaults channel from conda-lock
Overview
frictionless>=4.4,<5
tofrictionless>=5,<6
across the board #3293Testing
make pytest-coverage
locally in the new environment, usingferc-xbrl-extractor>=1.4.0
and everything seemed to work.To-do list