search

catalyst-cooperative / pudl

The Public Utility Data Liberation Project provides analysis-ready energy system data to climate advocates, researchers, policymakers, and journalists.
https://catalyst.coop/pudl
MIT License
456 stars 106 forks source link

Remove logic related to the ferc1 testing database #342

Closed zaneselvans closed 4 years ago

zaneselvans commented 5 years ago

With implementation of #320 and #326 and other python test and packaging related issues, there's no longer really a need for the ferc1_test database. Since #283 it has become an SQLite database, which is just a file on disk, and the location of that file is now changed dynamically for the test runs -- typically going into a temporary directory created by pytest -- so having a separate testing output is now an unnecessary complication.

exalate-issue-sync[bot] commented 4 years ago

Zane Selvans commented: This was surprisingly easy! The only snag is that now if you tell it you want to use the live FERC DB, you also need to tell it where to find that DB – i.e. “live” means “already exists” (as opposed to non-testing), so it needs to be paired with {{--pudl_in=AUTO}} or something like that.