UCL / UCL/STIR

zoom.cxx and origin

Open
#254 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
160
Forks
113
Avg merge
12d 15h
Merged PRs (30d)
1

Description

When using even-sized images, there's confusion about how zoom_image works (when specifying zoom factors, as opposed to an image), as it attempts to zoom around the "centre" of the image. This is somewhat described in the FAQ.

this won't be possible anymore once we support non-standard bed positions etc, see #223.

Note: changing this will break backwards compatibility.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the C++ implementation and tests for zoom_image, then read the linked FAQ explanation of the STIR coordinate system and issue #223. Determine the intended origin for zoom factors on even-sized images and document or test the expected behavior, noting that the change must account for non-standard bed positions and breaks backwards compatibility.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-vision
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.