Closed Vismayak closed 2 months ago
Contacted Nathanael and I believe the last step was updating dislocation depending on hoice dislocation and unsafe occupancy. I think last step would be figuring out whether to modify semantics for current popdislocation dataframe or create a different version
Update- Update in this PR
Should we always add all the new dislocation columns even if the new dislocation isn't selected (but empty)? @longshuicy @Vismayak ?
@navarroc could you please check the PR again, I have added the columns to be permanent even if flags are not called and to have boolean values.
Should unsafe_occupancy be an empty column instead of initialized to False if we are not setting choice_dislocation?
Also, we may want to trim the column name to unsafe_occ so it fits into 10 character limit if joining with a shapefile.
Sorry, my bad, the test is setting unsafe_occupancy value, ignore my first statement, but the length of the column is still a question
We may also want to shorten choice_dislocation to choice_dis so it meets the 10 character limit for shapefiles.
Adding the requested changes from Nathanael to have flags that calculate choice_dislocation and unsafe occupancy