search

open-forest-observatory / geograypher

Multiview Semantic Reasoning with Geospatial Data
BSD 3-Clause "New" or "Revised" License
10 stars 4 forks source link

WIP: Add workflow convenience functions #99

Closed russelldj closed 3 months ago

russelldj commented 3 months ago

This closes #95 and closes #96

russelldj commented 3 months ago

@youngdjn No need to provide a detailed review, I just thought you might find this interesting.

russelldj commented 3 months ago

I tried adding the function definitions to the files that can be used as command line scripts and I think I like it. Now the script files are a bit more confusing, but you could look at them and see what code is being called. This method avoids creating additional files, which was my main motivation.

My main motivation for this was writing the reusable functions to use for scripted experiments. In that case, because of the __init__.py file definition, you could import either of these from geograypher.entrypoints directly. Maybe they should even be top-level imports at some point, probably along with the major class definitions.