ros-perception / ros-perception/image_common

republish 'node' functionality is broken in Jazzy

Open
#355 4 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
153
Forks
253
Avg merge
5m
Merged PRs (30d)
1

Description

I summarised the problem with republish in Jazzy here: https://github.com/ros-perception/image_common/pull/275#issuecomment-2838833712

in that merge the 'node' was replaced by 'component', so any existing use of it as node now fails silently. not a great way to change API

Contributor guide

No contributing guide indexed for this repository

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 with pull request #275 and the linked issue comment, then trace the republish functionality where the node argument was replaced by component. Reproduce the existing node use case in Jazzy and verify that it no longer fails silently while preserving the intended API behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api, robotics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.