republish 'node' functionality is broken in Jazzy

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

Nobody has claimed this yet.

Assessment

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

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.

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

Dominant language
C++
Stars
154
Forks
258
Avg merge
2d 1h
Merged PRs (30d)
3

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

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.

More from ros-perception/image_common

All issues in ros-perception/image_common

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.