Merged
Conversation
bpwilcox
pushed a commit
that referenced
this pull request
Sep 20, 2018
* adding map_server tests & starting port of costmap_2d package * adding integration test node * + integration tests * ++ integration test * create branch for tests
bpwilcox
pushed a commit
that referenced
this pull request
Sep 20, 2018
* adding map_server tests & starting port of costmap_2d package * adding integration test node * + integration tests * ++ integration test * create branch for tests
bpwilcox
pushed a commit
that referenced
this pull request
Sep 24, 2018
* Adding ported map_server, env_model, and env_model_msgs * Delete gridmap.h This file is unused / hasn't been ported to ROS2 yet * Map server tests (#1) * adding map_server tests & starting port of costmap_2d package * adding integration test node * + integration tests * create branch for tests * added license info, addressed formatting and white spaces, implemented tests etc. * restructure of map_server * additional directory refactor * clean up package * all test passing (cpplint, uncrustify, unit, integration, etc.) * added TODO for publisher * dropping yaml-cpp backwards support, changed service/topic name * addressing feedback: changed all fprint to RCLCPP_ERROR, split main and server class, fixed yaml-cpp path, throwing exceptions up from yaml parser, cleaned cmakelist, added gtest assert throws, fixes naming conventions, added namespace * changed class/file names, replaced fprints, took out redundant createMap, adhered to project cmakelist style and build_test, deleted unused packages * changed name integration to component test, passes unit, component, lint tests, uncrustify, etc.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
changing working directory for tests back to map_server-devel