Skip to content

Fix incorrect assignment operator in function argument - #136

Merged
PondiB merged 3 commits into
devfrom
copilot/sub-pr-134-again
Feb 17, 2026
Merged

Fix incorrect assignment operator in function argument#136
PondiB merged 3 commits into
devfrom
copilot/sub-pr-134-again

Conversation

Copilot AI commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

Addresses review feedback on #134 regarding incorrect R syntax in the france_example_02.R file.

Changes

  • Fixed spatial_extent <- list( to spatial_extent = list( in p$load_collection() call

In R, <- is for variable assignment while = is required for named function arguments:

# Before (incorrect)
datacube_crop <- p$load_collection(
  id = "sentinel-s2-l2a-cogs",
  spatial_extent <- list(  # Wrong operator
    west = ...,
    ...
  )
)

# After (correct)
datacube_crop <- p$load_collection(
  id = "sentinel-s2-l2a-cogs",
  spatial_extent = list(  # Correct operator
    west = ...,
    ...
  )
)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: PondiB <16879290+PondiB@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on examples update and refactor NDVI Fix incorrect assignment operator in function argument Feb 17, 2026
Copilot AI requested a review from PondiB February 17, 2026 14:41
@PondiB
PondiB marked this pull request as ready for review February 17, 2026 14:49
@PondiB
PondiB merged commit d39f48d into dev Feb 17, 2026
@PondiB
PondiB deleted the copilot/sub-pr-134-again branch February 17, 2026 14:49
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.

2 participants