USFS-PNW / Fia-Biosum-Manager

User interface and main code repository for Biosum
http://biosum.info/
Other
3 stars 3 forks source link

DATABASE: Consider renaming landclcd to cond_status_cd #286

Closed lbross closed 1 year ago

lbross commented 1 year ago

Need to evaluate where/how the fields are used in BioSum to determine scope. Landclcd is an artifact of legacy systems and is confusing for new users and those familiar with FIADB.

lbross commented 1 year ago

Landclcd is not used widely throughout BioSum. It is used when creating/populating the condition table and creating the FVSIn databases. Renaming it to cond_status_cd would not be difficult but does require version_control to update the field name in the master.mdb/cond table for existing projects. Cond_status_cd is used during the plot load process in work tables when calculating the adjustment factors. This would not change. I uploaded a document to box that describes the references to landclcd and the usage of cond_status_cd in the adjustment process. Holding on this for now because it is not critical path.

sebbusby commented 1 year ago

master.cond.landclcd field is correctly being renamed to cond_status_cd