Get SCANFI species layers without Google authentication#191
Open
eliotmcintire wants to merge 2 commits into
Open
Get SCANFI species layers without Google authentication#191eliotmcintire wants to merge 2 commits into
eliotmcintire wants to merge 2 commits into
Conversation
Use reproducible::listGoogleDriveFolder() in place of a direct googledrive::drive_ls() call. When a directory-remap manifest is set via options(reproducible.urlRemap = ...) (e.g. from buckethost::makeMirrorManifest(directories = TRUE)), the SCANFI species layers are enumerated from a public mirror with no Google authentication -- the immediate use case is training/workshops. With no manifest set it falls back to drive_ls() and authenticates exactly as before. The folder listing's `url` column (mirror URL when remapped, else the Drive file URL) is used directly, since the mirror listing carries no Drive id. Requires the companion reproducible change adding listGoogleDriveFolder() and directory remaps. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The folder-listing swap in loadSCANFISpeciesLayers depends on reproducible::listGoogleDriveFolder() and directory-remap support, added in reproducible 3.1.1.9058 (PR PredictiveEcology/reproducible#520). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Dependency is now concrete: requires reproducible (>= 3.1.1.9058), added in PredictiveEcology/reproducible#520 ( |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
loadSCANFISpeciesLayers()now lists the SCANFI species-layer folder viareproducible::listGoogleDriveFolder()instead of callinggoogledrive::drive_ls()directly.When a directory-remap manifest is set:
the SCANFI files are enumerated from a public mirror (Arbutus object store) with no Google authentication — no Google account, no
drive_auth(). The immediate use case is training/workshops, where participants can pull SCANFI layers anonymously.Why this is safe / backwards compatible
listGoogleDriveFolder()falls back togoogledrive::drive_ls()and authenticates exactly as before. Identical behaviour.prepInputs(), so only the folder listing needed changing.urlcolumn is used directly (mirror URL when remapped, else the Drive file URL) instead of buildinghttps://drive.google.com/file/d/<id>— because the mirror listing carries no Drive id.The change
Two lines in
R/maps.R(loadSCANFISpeciesLayers):Dependency note
reproduciblechange that addslistGoogleDriveFolder()and directory-remap support (thetype="dir"manifest column +byDir). That must be merged toreproducible@developmentbefore LandR CI here will pass. Once it lands, the existingRemotes: PredictiveEcology/reproducible@developmentpicks it up; areproducible (>=version floor bump inDESCRIPTIONcan follow when reproducible releases.Scope
Only the
loadSCANFISpeciesLayerssite (maps.R:1520). The otherdrive_lssites are intentionally left as-is (the kNN twin would also needshared_drivepassthrough; theprepSpeciesLayerssites navigate Drive subfolder structure, not file listings).🤖 Generated with Claude Code