[FEATURE] Support AWS regions
kevinrue opened this issue · 2 comments
Currently, paws.storage::s3() uses a single region specified in the default location~/.aws/config.
The iSEEindex dataset configuration files should support an optional field (e.g. region) where users can override the default region (or set one if none are defined yet).
Note: This might all be simpler to configure as a YAML file rather than a CSV file. However, let's not conflate the two updates in the same issue. I'll open another issue for the YAML.
Context
The precache() function is located here:
https://github.com/iSEE/iSEEindex/blob/main/R/iSEEindexResource-class.R#L317
To keep the native S3 URI unchanged, the only option is to store the region as an additional attribute of the iSEEindexS3Resource class.
This issue cannot be resolved until paws-r/paws#571 is resolved first.
Partly implemented in #26, with warning that 'region' metadata is still ignored if present in the resource metadata.
If this solution turns out to be too confusing, I can disable the functionality entirely (including warnings), while waiting for the bug fix to the issue linked above.