Skip to content

Changed path to StackedSpatRaster in .rda file#64

Merged
wkearn merged 1 commit intoTopoToolbox:mainfrom
ktlenz:fix_data
Feb 23, 2026
Merged

Changed path to StackedSpatRaster in .rda file#64
wkearn merged 1 commit intoTopoToolbox:mainfrom
ktlenz:fix_data

Conversation

@ktlenz
Copy link
Copy Markdown
Contributor

@ktlenz ktlenz commented Feb 20, 2026

This PR closes #63 by replacing the hard-coded path previously stored in srtm_bigtujunga30m_utm11.rda with a StackedSpatRaster object. The new data should now work across systems.

@wkearn wkearn self-requested a review February 23, 2026 08:05
Copy link
Copy Markdown
Member

@wkearn wkearn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @ktlenz!

This resolves #63, so I'll merge. It seems like the PackedSpatRaster is the right way to go here, but it might be more user-friendly to be able to unwrap the PackedSpatRaster into a GRIDobj so that something like

data(srtm_bigtujunga30m_utm11)
dem <- GRIDobj(srtm_bigtujunga30m_utm11)

works. Right now it seems that you need to convert it to a SpatRaster then a GRIDobj.

@wkearn wkearn merged commit e9f12fd into TopoToolbox:main Feb 23, 2026
3 checks passed
@ktlenz ktlenz deleted the fix_data branch February 23, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect path to Big Tujunga data

2 participants