Closed jsstevenson closed 3 months ago
probably a git crime to do all of these at once but they felt relevant
I think this is sufficient to close #204 for now. If we want to circle back and make it actually lazy -- initialize a fetch on instance creation and then await it when actually calling methods -- we can reopen.
DataDownload
class.check_status()
method to preview whether data dependencies can be acquired without errors (close #286). Includes current UTA_DB_URL in logging (close #287)FileNotFoundError
if user-provided file paths don't exist. raiseValueError
if they aren't filesdata
module toresources
importlib.resources
to acquire the transcript mappings file rather than relying on relative file structure (hypothetically more reliable for reasons I don't totally understand)As far as I can tell, this shouldn't impact anything in production, because the environment variables are the first thing
cst
checks -- these changes basically only affect what happens if those fail or aren't provided.