Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ign -> gz CMake, Python, Partial Source, and File Migrations : gz-physics #381

Merged
merged 2 commits into from
Jul 15, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions .github/ci/packages.apt
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
libbenchmark-dev
libeigen3-dev
libignition-cmake3-dev
libignition-common5-dev
libignition-math7-dev
libignition-math7-eigen3-dev
libignition-plugin2-dev
libignition-utils2-cli-dev
libignition-utils2-dev
libgz-cmake3-dev
libgz-common5-dev
libgz-math7-dev
libgz-math7-eigen3-dev
libgz-plugin2-dev
libgz-utils2-cli-dev
libgz-utils2-dev
libsdformat13-dev
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
uses: actions/checkout@v2
- name: Compile and test
id: ci
uses: ignition-tooling/action-ignition-ci@focal
uses: gazebo-tooling/action-gz-ci@focal
with:
codecov-enabled: true
cppcheck-enabled: true
Expand All @@ -24,4 +24,4 @@ jobs:
uses: actions/checkout@v2
- name: Compile and test
id: ci
uses: ignition-tooling/action-ignition-ci@jammy
uses: gazebo-tooling/action-gz-ci@jammy
2 changes: 1 addition & 1 deletion .github/workflows/pr-collection-labeler.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,6 @@ jobs:
steps:
- name: Add collection labels
if: github.event.action == 'opened'
uses: ignition-tooling/pr-collection-labeler@v1
uses: gazebo-tooling/pr-collection-labeler@v1
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
6 changes: 2 additions & 4 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,7 @@ find_package(gz-cmake3 REQUIRED)
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)

gz_configure_project(
REPLACE_IGNITION_INCLUDE_PATH gz/physics
VERSION_SUFFIX pre1)
gz_configure_project(VERSION_SUFFIX pre1)

#============================================================================
# Set project-specific options
Expand Down Expand Up @@ -96,7 +94,7 @@ set(GZ_PHYSICS_RESOURCE_DIR "${CMAKE_SOURCE_DIR}/resources")

# Plugin install dirs
set(GZ_PHYSICS_ENGINE_INSTALL_DIR
${CMAKE_INSTALL_PREFIX}/${IGN_LIB_INSTALL_DIR}/gz-${IGN_DESIGNATION}-${PROJECT_VERSION_MAJOR}/engine-plugins
${CMAKE_INSTALL_PREFIX}/${GZ_LIB_INSTALL_DIR}/gz-${GZ_DESIGNATION}-${PROJECT_VERSION_MAJOR}/engine-plugins
)

#============================================================================
Expand Down
4 changes: 2 additions & 2 deletions Changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
1. Fix TPE Link velocity not being updated and Model velocity not having any effect.
* [Pull request #289](https://github.com/gazebosim/gz-physics/pull/289)

1. Make ignition-physics CMake config files relocatable
1. Make gz-physics CMake config files relocatable
* [Pull request #282](https://github.com/gazebosim/gz-physics/pull/282)

1. Added DART feature for setting joint limits dynamically.
Expand Down Expand Up @@ -91,7 +91,7 @@
* [Pull request #287](https://github.com/gazebosim/gz-physics/pull/287)
* [Pull request #281](https://github.com/gazebosim/gz-physics/pull/281)

1. Make ignition-physics CMake config files relocatable
1. Make gz-physics CMake config files relocatable
* [Pull request #282](https://github.com/gazebosim/gz-physics/pull/282)

### Gazebo Physics 4.2.0 (2021-07-16)
Expand Down
2 changes: 1 addition & 1 deletion Migration.md
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ release will remove the deprecated code.

### Modifications

1. Depends on ignition-utils1.
1. Depends on gz-utils1.

1. Depends on sdformat11.

Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,12 +128,12 @@ Follow these steps to run tests and static code analysis in your clone of this r
Refer to the following table for information about important directories and files in this repository.

```
ign-physics
gz-physics
├── bullet Files for bullet plugin component.
├── dartsim Files for dartsim plugin component.
├── example Examples about how to use the library
├── heightmap Heightmap related header files.
├── include/ignition/physics Header files.
├── include/gz/physics Header files.
├── mesh Files for mesh component.
├── resources Model and mesh resource files used by tests.
├── sdf Files for sdf component.
Expand Down
4 changes: 2 additions & 2 deletions api.md.in
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
## Gazebo @IGN_DESIGNATION_CAP@
## Gazebo @GZ_DESIGNATION_CAP@

Gazebo @IGN_DESIGNATION_CAP@ is a component in Gazebo, a set of libraries
Gazebo @GZ_DESIGNATION_CAP@ is a component in Gazebo, a set of libraries
designed to rapidly develop robot and simulation applications.

## License
Expand Down
4 changes: 2 additions & 2 deletions bullet/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ target_link_libraries(${features} INTERFACE GzBullet::GzBullet)

gz_get_libsources_and_unittests(sources test_sources)

# TODO(MXG): Think about an gz_add_plugin(~) macro for ign-cmake
# TODO(MXG): Think about a gz_add_plugin(~) macro for gz-cmake
set(engine_name bullet-plugin)
gz_add_component(${engine_name}
SOURCES ${sources}
Expand All @@ -30,7 +30,7 @@ install(TARGETS ${bullet_plugin} DESTINATION ${GZ_PHYSICS_ENGINE_INSTALL_DIR})
# Install redirection headers
install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")

# The library created by `gz_add_component` includes the gz-physics version
# (i.e. libgz-physics1-name-plugin.so), but for portability,
Expand Down
26 changes: 0 additions & 26 deletions configure.bat

This file was deleted.

4 changes: 2 additions & 2 deletions dartsim/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@ endif()

install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")

gz_get_libsources_and_unittests(sources test_sources)

# TODO(MXG): Think about an gz_add_plugin(~) macro for ign-cmake
# TODO(MXG): Think about a gz_add_plugin(~) macro for gz-cmake
set(engine_name dartsim-plugin)
gz_add_component(${engine_name}
SOURCES ${sources}
Expand Down
2 changes: 1 addition & 1 deletion dartsim/src/CustomHeightmapShape.hh
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ namespace physics {
namespace dartsim {

/// \brief This class creates a custom derivative of dartsim's HeightmapShape
/// class which allows an gz::common::Heightmap to be converted into a
/// class which allows a gz::common::Heightmap to be converted into a
/// HeightmapShape that can be used by dartsim.
/// Using float precision because Bullet's collision detector doesn't support
/// double. common::HeightmapData also holds floats.
Expand Down
2 changes: 1 addition & 1 deletion dartsim/src/CustomMeshShape.hh
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ namespace physics {
namespace dartsim {

/// \brief This class creates a custom derivative of dartsim's MeshShape class
/// which allows an gz::common::Mesh to be converted into a MeshShape that
/// which allows a gz::common::Mesh to be converted into a MeshShape that
/// can be used by dartsim.
class CustomMeshShape : public dart::dynamics::MeshShape
{
Expand Down
6 changes: 3 additions & 3 deletions dartsim/src/JointFeatures_TEST.cc
Original file line number Diff line number Diff line change
Expand Up @@ -834,7 +834,7 @@ TEST_F(JointFeaturesFixture, JointAttachDetach)
// the same as it was before they were attached
fixedJoint->SetTransformFromParent(poseParentChild);

// The name of the link obtained using the ign-physics API should remain the
// The name of the link obtained using the gz-physics API should remain the
// same even though AttachFixedJoint renames the associated BodyNode.
EXPECT_EQ(bodyName, model2Body->GetName());

Expand All @@ -855,7 +855,7 @@ TEST_F(JointFeaturesFixture, JointAttachDetach)
// now detach joint and expect model2 to start moving again
fixedJoint->Detach();

// The name of the link obtained using the ign-physics API should remain the
// The name of the link obtained using the gz-physics API should remain the
// same even though Detach renames the associated BodyNode.
EXPECT_EQ(bodyName, model2Body->GetName());

Expand Down Expand Up @@ -1030,7 +1030,7 @@ TEST_F(JointFeaturesFixture, JointAttachDetachSpawnedModel)
const auto poseParent = dartBody1->getTransform();
const auto poseChild = dartBody2->getTransform();

// Before ign-physics PR #31, uncommenting the following `step` call makes
// Before gz-physics PR #31, uncommenting the following `step` call makes
// this test pass, but commenting it out makes it fail.
// world->Step(output, state, input);
auto fixedJoint = model2Body->AttachFixedJoint(model1Body);
Expand Down
2 changes: 1 addition & 1 deletion dartsim/src/SDFFeatures.cc
Original file line number Diff line number Diff line change
Expand Up @@ -665,7 +665,7 @@ Identity SDFFeatures::ConstructSdfLink(
this->ConstructSdfCollision(linkIdentity, *collision);
}

// ign-physics is currently ignoring visuals, so we won't parse them from the
// gz-physics is currently ignoring visuals, so we won't parse them from the
// SDF
// for (std::size_t i = 0; i < _sdfLink.VisualCount(); ++i)
// {
Expand Down
2 changes: 1 addition & 1 deletion heightmap/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ target_link_libraries(${heightmap}

install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")
2 changes: 1 addition & 1 deletion include/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
add_subdirectory(gz)
install(DIRECTORY ignition DESTINATION ${IGN_INCLUDE_INSTALL_DIR_FULL})
install(DIRECTORY ignition DESTINATION ${GZ_INCLUDE_INSTALL_DIR_FULL})
2 changes: 1 addition & 1 deletion include/gz/physics/TemplateHelpers.hh
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ namespace gz
/// \brief Use this macro to create an API "selector" for a custom class.
///
/// Features may define APIs for class types that are not anticipated ahead of
/// time by the ign-physics library. When aggregating the API for that class
/// time by the gz-physics library. When aggregating the API for that class
/// from a set of features, the Aggregator must be given a Selector that can
/// ignore features that don't mention the class (or else a compilation failure
/// would occur).
Expand Down
8 changes: 4 additions & 4 deletions include/gz/physics/detail/RelativeQuantity.hh
Original file line number Diff line number Diff line change
Expand Up @@ -477,8 +477,8 @@ namespace gz
const Quantity &_box,
const RotationType &/*_currentCoordinates*/)
{
// TODO(anyone): Replace with gzwarn when/if we add an ign console
// dependency to ign-physics
// TODO(anyone): Replace with gzwarn when/if we add a gz console
// dependency to gz-physics
std::cerr << "[AABBSpace::ResolveToWorldCoordinates] Warning: "
<< "Axis-aligned bounding boxes cannot undergo coordinate "
<< "changes.\n";
Expand All @@ -490,8 +490,8 @@ namespace gz
const RotationType &/*_currentCoordinates*/,
const RotationType &/*_targetCoordinates*/)
{
// TODO(anyone): Replace with gzwarn when/if we add an ign console
// dependency to ign-physics
// TODO(anyone): Replace with gzwarn when/if we add a gz console
// dependency to gz-physics
std::cerr << "[AABBSpace::ResolveToTargetCoordinates] Warning: "
<< "Axis-aligned bounding boxes cannot undergo coordinate "
<< "changes.\n";
Expand Down
2 changes: 1 addition & 1 deletion mesh/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ target_link_libraries(${mesh}

install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")
2 changes: 1 addition & 1 deletion sdf/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ target_link_libraries(${sdf} INTERFACE ${SDFormat_LIBRARIES})

install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")
2 changes: 1 addition & 1 deletion sdf/include/gz/physics/sdf/ConstructVisual.hh
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class ConstructSdfVisual : public virtual Feature
class Link : public virtual Feature::Link<PolicyT, FeaturesT>
{
// TODO(MXG): Return a Shape type instead of a bool once we have shape
// features in the core ign-physics library.
// features in the core gz-physics library.
public: bool ConstructVisual(const ::sdf::Visual &_visual);
};

Expand Down
2 changes: 1 addition & 1 deletion test/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#============================================================================
# Do a fake install of ign-physics in order to test the examples.
# Do a fake install of gz-physics in order to test the examples.
#============================================================================
set(FAKE_INSTALL_PREFIX "${CMAKE_BINARY_DIR}/fake/install")

Expand Down
2 changes: 1 addition & 1 deletion test/MockCenterOfMass.hh
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
namespace mock
{
/////////////////////////////////////////////////
// TODO(MXG): Offer an gz::physics::Vector class that accepts a
// TODO(MXG): Offer a gz::physics::Vector class that accepts a
// FeaturePolicy type.
template <typename FeaturePolicyT>
using Vector = Eigen::Matrix<
Expand Down
2 changes: 1 addition & 1 deletion test/benchmark/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include(IgnBenchmark)
include(GzBenchmark)

set(tests
ExpectData.cc
Expand Down
4 changes: 2 additions & 2 deletions test/integration/FrameSemantics.hh
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@
*
*/

#ifndef IGNITION_PHYSICS_TEST_INTEGRATION_FRAMESEMANTICS_HH
#define IGNITION_PHYSICS_TEST_INTEGRATION_FRAMESEMANTICS_HH
#ifndef GZ_PHYSICS_TEST_INTEGRATION_FRAMESEMANTICS_HH
#define GZ_PHYSICS_TEST_INTEGRATION_FRAMESEMANTICS_HH

#include <gtest/gtest.h>
#include <cmath>
Expand Down
4 changes: 2 additions & 2 deletions test/integration/JointTypes.hh
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@
*
*/

#ifndef IGNITION_PHYSICS_TEST_INGEGRATION_JOINTTYPES_HH
#define IGNITION_PHYSICS_TEST_INGEGRATION_JOINTTYPES_HH
#ifndef GZ_PHYSICS_TEST_INGEGRATION_JOINTTYPES_HH
#define GZ_PHYSICS_TEST_INGEGRATION_JOINTTYPES_HH

#include <gtest/gtest.h>
#include <string>
Expand Down
2 changes: 1 addition & 1 deletion test/performance/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
gz_get_sources(tests)

# ExpectData test causes lcov to hang
# see ign-cmake issue 25
# see gz-cmake issue 25
if("${CMAKE_BUILD_TYPE_UPPERCASE}" STREQUAL "COVERAGE")
list(REMOVE_ITEM tests
ExpectData.cc)
Expand Down
2 changes: 1 addition & 1 deletion tpe/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ add_subdirectory(plugin)

install(
DIRECTORY include/
DESTINATION "${IGN_INCLUDE_INSTALL_DIR_FULL}")
DESTINATION "${GZ_INCLUDE_INSTALL_DIR_FULL}")
2 changes: 1 addition & 1 deletion tpe/plugin/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ target_include_directories(${features} SYSTEM INTERFACE)

gz_get_libsources_and_unittests(sources test_sources)

# TODO(MXG): Think about an gz_add_plugin(~) macro for ign-cmake
# TODO(MXG): Think about a gz_add_plugin(~) macro for gz-cmake
set(engine_name tpe-plugin)
gz_add_component(${engine_name}
SOURCES ${sources}
Expand Down
2 changes: 1 addition & 1 deletion tpe/plugin/src/SDFFeatures.cc
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,7 @@ Identity SDFFeatures::ConstructSdfCollision(
}
// \todo(anyone) add mesh. currently mesh has to be loaded externally
// and passed in as argument as there is no logic for searching resources
// in ign-physics
// in gz-physics
const auto collisionIdentity = this->AddCollision(link->GetId(), *collision);

// set collide bitmask
Expand Down
4 changes: 2 additions & 2 deletions tutorials.md.in
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
\page tutorials Tutorials

Welcome to the Gazebo @IGN_DESIGNATION_CAP@ tutorials. These tutorials
Welcome to the Gazebo @GZ_DESIGNATION_CAP@ tutorials. These tutorials
will guide you through the process of understanding the capabilities of the
Gazebo @IGN_DESIGNATION_CAP@ library and how to use the library effectively.
Gazebo @GZ_DESIGNATION_CAP@ library and how to use the library effectively.


**Contents**
Expand Down
2 changes: 1 addition & 1 deletion tutorials/01_intro.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ Gazebo Physics extensibility and modularity.
For a big picture of the Gazebo Physics operation in Gazebo ecosystem, see
the abstract diagram below:

@image html img/ign-libraries.png
@image html img/gz-libraries.png

In general, `gz-sim` is the main simulation library, in which its
functionalities are powered by many component libraries.
Expand Down
Loading