Skip to content

Commit

Permalink
Partial source and cmake migrations
Browse files Browse the repository at this point in the history
Signed-off-by: methylDragon <[email protected]>
  • Loading branch information
methylDragon committed Jul 13, 2022
1 parent 360b742 commit 6018890
Show file tree
Hide file tree
Showing 35 changed files with 73 additions and 101 deletions.
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
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ 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
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 an 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 an gz_add_plugin(~) macro for gz-cmake
set(engine_name dartsim-plugin)
gz_add_component(${engine_name}
SOURCES ${sources}
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/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 an 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

0 comments on commit 6018890

Please sign in to comment.