Pass sources to DataSource.from_id()

[e-belfer]

Overview

Relevant to catalyst-cooperative/pudl-archiver#506.

What problem does this address?

In #4003 I updated DataSource.dict_from_id() to take sources. However, from_id() is an upstream method called in the pudl-archiver repository, and I neglected to add sources as a parameter here. This PR fixes this problem.

What did you change?

• Added sources as a parameter to DataSource.from_id()
• Made SOURCES the default in from_id() and not dict_from_id(). from_id() is more widely called both in PUDL and the pudl-archiver repo, and this removes duplicate defaults in the class.

Documentation

Make sure to update relevant aspects of the documentation.

Tasks

Preview Give feedback

1. Update the release notes: reference the PR and related issues.
2. Update relevant Data Source jinja templates (see docs/data_sources/templates).
3. Update relevant table or source description metadata (see src/metadata).
4. Review and update any other aspects of the documentation that might be affected by this PR.

Testing

How did you make sure this worked? How can a reviewer verify this?

To-do list

Preview Give feedback

1. If updating analyses or data processing functions: make sure to update or write data validation tests (e.g. test_minmax_rows()).
2. Run make pytest-coverage locally to ensure that the merge queue will accept your PR.
3. Review the PR yourself and call out any questions or issues you have.
4. For minor ETL changes or data additions, once make pytest-coverage passes, make sure you have a fresh full PUDL DB downloaded locally, materialize new/changed assets and all their downstream assets and run relevant data validation tests using pytest and --live-dbs.
5. For bigger ETL or data changes run the full ETL locally and then run the data validations using make pytest-validate.
6. Alternatively, run the build-deploy-pudl GitHub Action manually.

[zschira]

Looks good!