Skip to content

noop#7

Draft
Copilot wants to merge 7 commits intomainfrom
copilot/add-offset-list-support
Draft

noop#7
Copilot wants to merge 7 commits intomainfrom
copilot/add-offset-list-support

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

  • Inspect existing offset behavior in profuseMultiImageFound2Fit, profuseFound2Fit, and ProFit profitSetupData
  • Add offset_list argument support to profuseMultiBandFound2Fit with safe defaults and length normalization
  • Propagate per-band offsets into each profitSetupData call in multi-band setup
  • Keep existing crop/region behavior unchanged when offset_list is not supplied
  • Update man/profuseMultiBand.Rd usage and arguments to document offset_list
  • Update related vignette wording that previously stated there was no multi-band offset support
  • Address review feedback by clarifying vignette wording and making list recycling explicit
  • Add offset_list type validation and clean indentation in function signature
  • Reject invalid numeric offset_list inputs that are not list-based per-band vectors
  • Validate each non-NULL offset element is numeric length 2
  • Clarify documentation that non-singleton offset_list must match image_list length
  • Run available validation/check commands in this environment and report results

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