Matt
Matt
Try again next release.
Thanks @kleintom > I changed all ST_Contains to ST_Covers I feel this is a safe assumption. > which should be used in new code? This is essentially everything but me...
A little review/thoughts on the present state: IIRC the strategy is to land this PR then do a followup where we merge everything down to the single `geography` column, correct?...
> in the gz case the Descendants and Exact options don't apply (at least not yet), so there's only the spatial option. Let's keep this basis for the original implementation...
@kleintom Peeking back at this. Let me know if I can help out in some regard. Again, not critical, just seeing if we can facilitate. Thanks!
@jlpereira When you have time can you do another review of the JS side of things please?
@kleintom I talked with some folks here, where going to try and get a couple different shape-files to experiment with, some simpler, some complicated. More feedback in general coming soon.
> migrate/20220927154908_add_cached_total_area_to_geographic_item.rb This is a bit of a problem. We very strongly don't want to go back and alter migrations, however here I think we could try to add the...
@kleintom Don't worry about pausing development, keep tacking on changes (date-line) if you wish.
@kleintom The blocker to us getting to Rails 7.2 has landed (https://github.com/rgeo/activerecord-postgis-adapter/pull/405#issuecomment-2455646003 ). We're still seeing issues on our test PR: https://github.com/SpeciesFileGroup/taxonworks/actions/runs/11672461015/job/32501118816, but I haven't poked at what might be...