search

weecology / retriever

Quickly download, clean up, and install public datasets into a database management system
http://data-retriever.org
Other
307 stars 132 forks source link

FIA dataset - 3 states are missing: NewMexico, Wyoming and Mississippi #176

Closed SteveViss closed 10 years ago

SteveViss commented 10 years ago

However, those states have been nationally standardized. I checked the data in order to get in when that has been did:

Where PLOT.DESIGNCD = 1, we have:

State_id PLOT.INVYR
MS 2006
NM 1999
WY 2000
ethanwhite commented 10 years ago

Thanks for the bug report Steve!

Those states were missing from our lists of states with nationally standardized sampling. Not sure why they got missed when we wrote the original script - my apologies. I'm working on the fix now.

ethanwhite commented 10 years ago

OK, there's a fix in my fork (https://github.com/ethanwhite/retriever/commit/704fcaa7fd96a48d3d6c389db0983f655666c897). I'm testing it locally now and will get it in as soon as I confirm that it doesn't cause any hiccups.

SteveViss commented 10 years ago

Thanks Ethan!

ethanwhite commented 10 years ago

@SteveViss - just wanted to let you know that due to some bad computer juju today the install to run this patch is still finishing up. It should be done in the morning and barring any issues we'll get it into master shortly thereafter.

ethanwhite commented 10 years ago

@SteveViss - this should be all fixed now. You don't need to update the Retriever itself, just get the updated script. If you're using the GUI it should update automatically. If you're using the CLI just run retriever update and that should take care of it.

SteveViss commented 10 years ago

Thanks for your efficiency weecology team, really appreciated !

ethanwhite commented 10 years ago

Our pleasure. Thanks for reporting the bug! Working together like this is how we get nice things.