Skip to content

Add python headers for python bindings#788

Merged
wjwwood merged 1 commit intoros-visualization:hydro-develfrom
NikolausDemmel:patch-1
Jun 30, 2014
Merged

Add python headers for python bindings#788
wjwwood merged 1 commit intoros-visualization:hydro-develfrom
NikolausDemmel:patch-1

Conversation

@NikolausDemmel
Copy link
Copy Markdown
Contributor

Adding ${PYTHON_INCLUDE_DIRS} to the sip python bindings.

Needed at least on OSX/10.9 to compile successfully.

This should maybe be also ported to groovy and indigo.

Adding ${PYTHON_INCLUDE_DIRS} to the sip python bindings.

Needed at least on OSX/10.9 to compile successfully.
wjwwood added a commit that referenced this pull request Jun 30, 2014
Add python headers for python bindings
@wjwwood wjwwood merged commit d5ce76f into ros-visualization:hydro-devel Jun 30, 2014
@NikolausDemmel
Copy link
Copy Markdown
Contributor Author

Possibly that was the wrong fix and the sip_helper should get patched instead, see ros-visualization/qt_gui_core#46

NikolausDemmel added a commit to NikolausDemmel/python_qt_binding that referenced this pull request Jul 1, 2014
This fixes ros-visualization/qt_gui_core#46 and ros-visualization/rviz#788; needed at least on OS X, likely some other platforms as well.
@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Jul 1, 2014

You mean you think that the ${PYTHON_INCLUDE_DIRS} should already be in ${catkin_INCLUDE_DIRS}?

@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Jul 1, 2014

That may be the case, at any rate this shouldn't be a problem if the fix is done upstream too. We can try undoing it once the other issue is addressed.

@NikolausDemmel
Copy link
Copy Markdown
Contributor Author

You mean you think that the ${PYTHON_INCLUDE_DIRS} should already be in ${catkin_INCLUDE_DIRS}?

Nope. See ros-visualization/python_qt_binding#12 for a proposed change to the sip_helper to add the PYTHON_INCLUDE_DIRS, such that each use of sip_helper does not need to do it.

It shouldn't do any harm though. We could revert it, but no rush.

@NikolausDemmel
Copy link
Copy Markdown
Contributor Author

ros-visualization/python_qt_binding#12 has been merged, so feel free to revert this any time (or not, doesn't really matter much).

@NikolausDemmel NikolausDemmel deleted the patch-1 branch July 2, 2014 17:33
130s pushed a commit to 130s/rviz that referenced this pull request Aug 21, 2024
* Remove TF filter from ImageTransportDisplay

Fixes ros-visualization#639

Only the CameraDisplay cares about TF data, so this change moves the TF filter logic into the CameraDisplay.

This allows users to view images with the ImageDisplay no matter what the fixed frame is set to.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
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