-
Notifications
You must be signed in to change notification settings - Fork 31
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
Error Reading LTDB Data: File Path Not Working #416
Comments
thanks for raising @shengxiangjinuci I havent used the LTDB data in awhile. The issue here is our ltdb reader is hardcoded to an older version of the data and it expects all CSV files are in the root of the zip archive. The newest release has a slightly different structure (the files are inside a second subdirectory and have a few tweaks to filenames) ![]() this should be a simple fix in the code. I don't think its feasible to have compatibility with multiple versions of LTDB because we'll probably always be hardcoding the file paths, but maybe we could check the hashes of the zip files as a simple 'version' check on the ltdb |
alternatively we could have users pass individual paths to each csv file, but I think I prefer the zip approach |
The zip method is undoubtedly ideal, but perhaps allowing a list of each csv file to be passed would also be a good backup plan. |
Hi, I was wondering if it would be possible to share old versions of the LTDB file zip so that we can use this data directly in functions. Since I want to combine it with ACS data to create a long dataframe, I would like to do all the operations in geosnap for consistency. Best, |
@shengxiangjinuci i dont think i have an old version of LTDB (though I'll check) but the latest commit in #417 updates the paths to the newest data in the zip files (just released on their end) and also lets you pass a set of file paths in case you want to unpack the zips yourself. The latter approach should work with any version of the LTDB data unless they change the formatting of the CSV files sometime in the future |
Thanks, I've tried the updated version and it works great. |
Thanks for this wonderful tool.
I’m encountering an issue when trying to read the LTDB data using the gio.store_ltdb() function. Specifically, when I provide the file path to the zip file, the function does not work as expected.
Best,
The text was updated successfully, but these errors were encountered: