search

GenomicMedLab / cool-seq-tool

https://coolseqtool.readthedocs.io
MIT License
4 stars 0 forks source link

`SeqRepoAccess` should support `SeqRepoRESTDataProxy` #307

Open korikuzma opened 2 months ago

korikuzma commented 2 months ago

Feature description

Currently, SeqRepoAccess inherits from SeqRepoDataProxy (local). I think we should extend support for SeqRepoRESTDataProxy.

Use case

We're currently limited to using local SeqRepo. It would be nice to be able to have the option to use the REST dp

Proposed solution

SeqRepoAccess should inherit _DataProxy

Alternatives considered

No response

Implementation details

No response

Potential Impact

Provides more flexibility. We could also leverage REST for CI tests

Additional context

No response

Contribution

None

jsstevenson commented 2 months ago

👍 on this. SeqRepo uses a basic factory method to dispatch out to local vs REST proxies based on the structure of the URI, so we'll presumably need to do something similar.