Test to see if we can use all `prioritizr` has to offer with our patches method

emlab-ucsb / patchwise

Supplementary package to `oceandatr` for when entire "patches" of a feature should be protected

https://emlab-ucsb.github.io/patchwise/

0 stars 0 forks source link

Test to see if we can use all `prioritizr` has to offer with our patches method #8

Open echelleburns opened 9 months ago

echelleburns commented 9 months ago

Test all of the add_* functions in prioritizr to see what we can and cannot run with the patchwise framework.

Make a list of things that work/don't work and add them to the readme for users to easily see.

echelleburns commented 9 months ago

Constraints

add_locked_in_constraints()

locked in constraints must be specified in patchwise::create_patch_df() and patchwise::features_targets() instead of as add_locked_in_constraints()
see vignette for more details and examples

add_locked_out_constraints()

locked out constraints must be specified in patchwise::create_patch_df() and patchwise::features_targets() instead of as add_locked_out_constraints()
see vignette for more details and examples

echelleburns commented 9 months ago

It looks like a lot of this functionality breaks with patchwise because of the way we specify the patches. We will need to explore this further, but it might be a future issue. For now, we should make it clear in the documentation that constraints are built into the targets portion of the prioritization when users utilize patchwise