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

Release 2022.11.XX #1681

Merged
merged 1,463 commits into from
Nov 29, 2022
Merged

Release 2022.11.XX #1681

merged 1,463 commits into from
Nov 29, 2022

Conversation

zaneselvans
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Jun 9, 2022

Codecov Report

Base: 83.9% // Head: 85.1% // Increases project coverage by +1.1% 🎉

Coverage data is based on head (812b5b8) compared to base (f6c670c).
Patch coverage: 90.0% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1681     +/-   ##
=======================================
+ Coverage   83.9%   85.1%   +1.1%     
=======================================
  Files         64      72      +8     
  Lines       6879    8202   +1323     
=======================================
+ Hits        5775    6981   +1206     
- Misses      1104    1221    +117     
Impacted Files Coverage Δ
src/pudl/metadata/resources/ferc1.py 100.0% <ø> (ø)
src/pudl/metadata/resources/ferc714.py 100.0% <ø> (ø)
src/pudl/metadata/resources/glue.py 100.0% <ø> (ø)
src/pudl/metadata/resources/pudl.py 100.0% <ø> (ø)
src/pudl/metadata/sources.py 100.0% <ø> (ø)
src/pudl/output/censusdp1tract.py 87.0% <ø> (-2.6%) ⬇️
src/pudl/output/eia860.py 69.8% <ø> (+3.3%) ⬆️
src/pudl/output/eia923.py 99.3% <ø> (+2.5%) ⬆️
src/pudl/output/epacems.py 77.5% <ø> (-8.7%) ⬇️
src/pudl/output/ferc1.py 100.0% <ø> (ø)
... and 89 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@zaneselvans zaneselvans linked an issue Jun 9, 2022 that may be closed by this pull request
8 tasks
@zaneselvans zaneselvans changed the title First continuous deployment release Release 2022.06.XX Jun 19, 2022
@zaneselvans zaneselvans changed the title Release 2022.06.XX Release 2022.07.XX Jul 8, 2022
@zaneselvans zaneselvans changed the title Release 2022.07.XX Release 2022.08.XX Aug 2, 2022
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@zaneselvans zaneselvans changed the title Release 2022.08.XX Release 2022.10.XX Oct 6, 2022
cmgosnell and others added 22 commits October 31, 2022 12:56
updates:
- [github.com/pre-commit/mirrors-prettier: v3.0.0-alpha.3 → v3.0.0-alpha.4](pre-commit/mirrors-prettier@v3.0.0-alpha.3...v3.0.0-alpha.4)
- [github.com/asottile/pyupgrade: v3.1.0 → v3.2.0](asottile/pyupgrade@v3.1.0...v3.2.0)
Updates the requirements on [dask](https://github.com/dask/dask) to permit the latest version.
- [Release notes](https://github.com/dask/dask/releases)
- [Changelog](https://github.com/dask/dask/blob/main/docs/release-procedure.md)
- [Commits](dask/dask@2021.08.0...2022.10.2)

---
updated-dependencies:
- dependency-name: dask
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…te-config

[pre-commit.ci] pre-commit autoupdate
…/dask-gte-2021.8-and-lt-2022.10.3

Update dask requirement from <2022.10.1,>=2021.8 to >=2021.8,<2022.10.3
Progress on #2012 #2014

* Fixed a bug in how the DBF row numbers that need to be mapped are
  identified. Now it looks for any time the row_literal associated with
  a row number has changed from one year to the next, rather than
  selecting the first instance of each distinct combination of
  row_literal and row_number.
* Also discovered that there's an obscure row_status field that
  differentiates between annual (A) and quarterly (Q) row literals, and
  is part of the f1_row_lit_tbl primary key, but it only shows up in
  association with the f1_schedules_list table. I integrated it but...
  maybe that table should just be excluded from the row mapping
  template?
* Added some (janky) helper functions to pudl.transform.ferc1 to manage
  the generation of the row maps. This location is temporary. They
  should probably become methods of a Ferc1 abstract transformer class
  for reshaped tables, or maybe end up in a different module. Not sure
  how they'll end up getting used yet though.
* Updated the dbf_to_xbrl.csv file to include all of the possible rows
  that could need mapping (4270 in total).
* Removed the XBRL specific metadata fields from the dbf_to_xbrl.csv
  file, since they should (hopefully) be available programmatically from
  the metadata @zschira is extracting from the XBRL taxonomies, and can
  be joined to this table based on the xbrl_column_stem.
* Updated the plant_in_service transform to use the new row map. Need to
  test on all of the years.
@zaneselvans zaneselvans linked an issue Nov 18, 2022 that may be closed by this pull request
33 tasks
zaneselvans and others added 5 commits November 18, 2022 17:05
Updates the requirements on [dask](https://github.com/dask/dask) to permit the latest version.
- [Release notes](https://github.com/dask/dask/releases)
- [Changelog](https://github.com/dask/dask/blob/main/docs/release-procedure.md)
- [Commits](dask/dask@2021.08.0...2022.11.1)

---
updated-dependencies:
- dependency-name: dask
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…/dask-gte-2021.8-and-lt-2022.11.2

Update dask requirement from <2022.11.1,>=2021.8 to >=2021.8,<2022.11.2
After re-running the full ETL locally, I found that the new row counts
matched the ones that were being found by the nightly builds (which were
different from what I was finding prior to rebuilding my DB).  They were
also the row counts that Christina was finding in her DB. So...
hopefully they are actually correct!

If they once again fail to match up, then maybe we should look at:

* Are the contents of the `fuel_ferc1` or `fbp_ferc1` tables being
  altered somewhere in the course of the validation tests, such that
  they have a different number of rows when they're checked in isolation
  vs. when they're checked as part of the overall data validations?

* Is the ETL being run with code from the commit that we think it's
  being run from? Or is it somehow checking out a different version than
  we expect?
bendnorman and others added 7 commits November 22, 2022 13:48
Updates the requirements on [pandas](https://github.com/pandas-dev/pandas) to permit the latest version.
- [Release notes](https://github.com/pandas-dev/pandas/releases)
- [Changelog](https://github.com/pandas-dev/pandas/blob/main/RELEASE.md)
- [Commits](pandas-dev/pandas@v1.4.0...v1.5.2)

---
updated-dependencies:
- dependency-name: pandas
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…-records

Address missing fuel ferc1 records

I'm going to merge this into `dev` without letting the CI finish since I'm going to immediately run the CI on `dev`...
@zaneselvans
Copy link
Member Author

@bendnorman @cmgosnell Your two small PRs: #2079 & #2091 seem like they could conceivably be merged in here before the release... do we want to risk that? Or not?

…/pandas-gte-1.4-and-lt-1.5.3

Update pandas requirement from <1.5.2,>=1.4 to >=1.4,<1.5.3
Add loglevel and logfile to all CLI commands
@zaneselvans
Copy link
Member Author

Okay I've merged in @bendnorman's logging standardization, and am holding off on the 860m for the moment. Was thinking I would kick off the nightly build job.

@zaneselvans zaneselvans merged commit ba4a3ad into main Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PUDL Release v2022.11.30
5 participants