search

OpenTreeOfLife / otindex

opentree index using postgres and pyramid
BSD 2-Clause "Simplified" License
0 stars 0 forks source link

Fixes to add_update #56

Closed snacktavish closed 2 years ago

snacktavish commented 2 years ago

I didn't carefully dig into why these were necessary - maybe python3 related? But they do work. (currently deployed on production)

jimallman commented 2 years ago

I recognize the change in string-types as a common python 2-to-3 update. I do like removing the recursive otindex from requirements files. To test this properly, we should remove the already-deployed otindex from inside the venv (it might be a symlink to a repo folder alongside).

snacktavish commented 2 years ago

Makes sense. I am updating the ansible deploy to do a better job on dev (related to https://github.com/OpenTreeOfLife/peyotl/pull/180#discussion_r988409176) and the we can properly try it out there!

(also I accidentally put unconnected changes into the same commit :shrug: )

snacktavish commented 2 years ago

Pushed to devotindex, and is working well, but but didn't do a full clean reinstall yet.