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

Communication boiler #2114

Merged
merged 21 commits into from
Nov 19, 2023
Merged
Changes from 1 commit
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
Prev Previous commit
Next Next commit
Adding generation to makefile
  • Loading branch information
rishiso committed Oct 2, 2023
commit fadd539faefb48637b07aa9daa5cd4c9878e838a
7 changes: 7 additions & 0 deletions makefile
Original file line number Diff line number Diff line change
@@ -33,13 +33,19 @@ define cmake_build_target_perf
cd build-release-debug && cmake -GNinja -Wno-dev -DNO_WALL=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo $(CMAKE_FLAGS) --target -DBUILD_TESTS=ON .. && ninja $(NINJA_FLAGS) $1 install
endef

define communication_generation
cd soccer/src/soccer/strategy/agent/communication && python3 communication_gen.py
endef

all-perf:
$(call communication_generation, all)
$(call cmake_build_target_perf, all)
# perf (or "RelWithDebInfo"): almost as fast as release, some debug symbols
perf: all-perf

# used in GH Actions build-and-test
all:
$(call communication_generation, all)
$(call cmake_build_target, all)
# debug: slow executable, but many debug symbols (e.g. for GDB)
debug: all
@@ -57,6 +63,7 @@ release: all-release
# run if build-release-debug/ exists from a previous build
# and no CMake files or launch.py files have been changed
again:
$(call communication_generation, all)
(cd build-release-debug/ && ninja install)

# run soccer with default flags