Skip to content

Commit b683f16

Browse files
committed
clean comparison benchmarks
1 parent 005fdcd commit b683f16

File tree

6 files changed

+26
-304
lines changed

6 files changed

+26
-304
lines changed

CMakeLists.txt

-18
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,6 @@ if(WIN32 AND BUILD_SHARED_LIBS)
2121
endif()
2222

2323

24-
find_package(Reflexxes QUIET)
25-
26-
2724
add_library(ruckig
2825
src/ruckig/brake.cpp
2926
src/ruckig/position-step1.cpp
@@ -59,15 +56,6 @@ if(BUILD_ONLINE_CLIENT)
5956
endif()
6057

6158

62-
if(Reflexxes)
63-
set(REFLEXXES_TYPE "ReflexxesTypeII" CACHE STRING "Type of Reflexxes library") # or ReflexxesTypeIV
64-
message("Found Reflexxes ${REFLEXXES_TYPE}")
65-
66-
target_compile_definitions(ruckig PUBLIC WITH_REFLEXXES)
67-
target_link_libraries(ruckig PUBLIC Reflexxes::Reflexxes)
68-
endif()
69-
70-
7159
if(BUILD_EXAMPLES)
7260
foreach(example IN ITEMS 1_position 2_position_offline 5_velocity 6_stop 7_minimum_duration 8_dynamic_dofs)
7361
add_executable(example-${example} examples/${example}.cpp)
@@ -158,9 +146,6 @@ if(BUILD_TESTS)
158146
else()
159147
target_compile_options(test-target PRIVATE -Wall -Wextra)
160148
endif()
161-
if(Reflexxes)
162-
target_compile_definitions(test-target PUBLIC WITH_REFLEXXES)
163-
endif()
164149

165150
add_test(NAME test-target COMMAND test-target)
166151
endif()
@@ -169,8 +154,5 @@ endif()
169154
if(BUILD_BENCHMARK)
170155
add_executable(benchmark-target test/benchmark-target.cpp)
171156

172-
if(Reflexxes)
173-
target_compile_definitions(benchmark-target PUBLIC WITH_REFLEXXES)
174-
endif()
175157
target_link_libraries(benchmark-target PRIVATE ruckig)
176158
endif()

cmake/FindReflexxes.cmake

-19
This file was deleted.

include/ruckig/reflexxes_comparison.hpp

-175
This file was deleted.

src/ruckig/python.cpp

-12
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,6 @@
77

88
#include <ruckig/ruckig.hpp>
99

10-
#ifdef WITH_REFLEXXES
11-
#include <ruckig/reflexxes_comparison.hpp>
12-
#endif
13-
1410

1511
namespace py = pybind11;
1612
using namespace pybind11::literals; // to bring in the `_a` literal
@@ -188,12 +184,4 @@ limited by velocity, acceleration, and jerk constraints.";
188184
.def_readonly("accel", &Profile::accel)
189185
.def("__repr__", &Profile::to_string);
190186
#endif
191-
192-
#ifdef WITH_REFLEXXES
193-
py::class_<Reflexxes<DynamicDOFs>>(m, "Reflexxes")
194-
.def(py::init<size_t, double>(), "dofs"_a, "delta_time"_a)
195-
.def_readonly("degrees_of_freedom", &Reflexxes<DynamicDOFs>::degrees_of_freedom)
196-
.def_readonly("delta_time", &Reflexxes<DynamicDOFs>::delta_time)
197-
.def("update", &Reflexxes<DynamicDOFs>::update);
198-
#endif
199187
}

test/benchmark-target.cpp

-6
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@
44

55
#include <ruckig/ruckig.hpp>
66

7-
#ifdef WITH_REFLEXXES
8-
#include <ruckig/reflexxes_comparison.hpp>
9-
#endif
10-
117

128
using namespace ruckig;
139

@@ -112,8 +108,6 @@ int main() {
112108
std::cout << "Ruckig size: " << sizeof(Ruckig<3, true>) << " Byte" << std::endl;
113109
std::cout << "Trajectory<3> size: " << sizeof(Trajectory<3>) << " Byte" << std::endl;
114110

115-
// Main comparison
116111
const size_t DOFs {7};
117112
benchmark<DOFs, Ruckig<DOFs, true>>(n, number_trajectories);
118-
// benchmark<DOFs, Reflexxes<DOFs>>(n, number_trajectories);
119113
}

0 commit comments

Comments
 (0)