save_spatial functions creating wrong file path

[sarahLy9]

saving spatial data is adding unnecessary nesting for example:

sbf_set_main(spatial_dir)
sbf_set_sub("workup", rm = TRUE)
sbf_save_spatial(water_temperature_location_lco)

puts it here:

'~Poisson/Privileged - Spatial/Spatial/ufr-wct/2025/spatial/workup'

I need it here:

'~Poisson/Privileged - Spatial/Spatial/ufr-wct/2025/workup'
We know its spatial data because its in Privileged - Spatial/Spatial (which is already redundant enough) and the extra layers of folders is messing up processing with QGIS.

It looks like this is because of an automatic behaviour that gives it an enclosing folder based on its data type.

[joethorley]

It's built into subfoldr2 to save object types into their own folders ie output/data, output/tables, output/plots etc to avoid clashes among different object types with the same name and ensure each data type is handled correctly so I'm not sure it would be a good idea to get rid of the spatial folder created by subfoldr2. Could we drop the second Spatial from ~\Poisson\Privileged - Spatial\Spatial\... so it just becomes ~\Poisson\Privileged - Spatial\...?

[sarahLy9]

right now the sbf_save_spatials() function will for example save the output here
"~/Poisson/Privileged - Spatial/Spatial/ufr-wct/2025/spatial/exported"
But this code will not read in the spatial data in the next script
sbf_set_main(file.path(spatial_dir, "spatial/exported"))
sbf_load_spatials()
despite
> file.path(spatial_dir, "spatial/exported")
[1] "~/Poisson/Privileged - Spatial/Spatial/ufr-wct/2025/spatial/exported"

so maybe the correction needs to be in the sbf_load_spatials()

[joethorley]

I effectively save the spatial objects using this code

library(subfoldr2)

options(poisson.privileged = TRUE)

sbf_set_main(poisreport::spatial_directory())
sbf_set_sub("exported", rm = TRUE)

sbf_save_spatial(footprint)

so to load them you'll need to use

library(subfoldr2)

options(poisson.privileged = TRUE)

sbf_set_main(poisreport::spatial_directory())
sbf_set_sub("exported")

sbf_load_spatials()

I'm happy to do some other way if it makes sense.