You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sedona.apache.org by "jiayuasu (via GitHub)" <gi...@apache.org> on 2026/05/17 07:24:19 UTC

Re: [PR] [SEDONA-756] feat: raster Python serde and with_bands() support [sedona]

jiayuasu commented on PR #2956:
URL: https://github.com/apache/sedona/pull/2956#issuecomment-4469721395

   Hi @prantogg — heads up, I rebased this branch onto current master to resolve the conflict in `docs/tutorial/raster.md` that surfaced after #2954 merged. New head: `c52c18ae2f` (your authorship preserved).
   
   Resolution choices for the UDF section:
   
   - Kept the new `### Raster to scalar` / `### Raster to raster` split, the `with_bands()` example, and the in-db/out-db warning admonition from your PR.
   - Dropped the old "returning a raster isn't supported, use `RS_MakeRaster` workaround" paragraph that #2954's rewrite was still carrying — now obsolete with `with_bands()`.
   - Kept the rewrite's `## Performance` heading instead of reverting to `## Performance optimization`, since the rename was unrelated to this feature.
   - Removed a misleading `[SedonaRaster.with_bands()]` link that pointed to `RS_MakeRaster.md`; `with_bands()` is a Python method, not a SQL function with its own page.
   - Mirrored the same edits in `docs/tutorial/raster.zh.md` (which #2954 added and was previously out of sync with the English content).
   
   If you have local commits on top of `pranav/feature/raster-python-serde` you'll need to `git fetch && git reset --hard origin/pranav/feature/raster-python-serde` (or rebase your local work onto the new tip) before continuing. Sorry for the force-push.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org