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

Add vector map server to RuntimeManager #739

Merged
merged 4 commits into from
Aug 28, 2017

Conversation

andoh104
Copy link

@andoh104 andoh104 commented Jul 23, 2017

Status

PRODUCTION / DEVELOPMENT

Description

FIx VectorMap Server Call failed autowarefoundation/autoware_ai#940

Related PRs

Same Pull Request as #728 (It was cancelled. )
Re-based on new master branch and tested

Todos

  • Tests
  • Documentation

Not tested the VectorMap service

Steps to Test or Reproduce

  • Check "vector_map_server"

screenshot from 2017-07-14 08 00 04

  • Then this error will not occur.
    vectormap_filtering: VectorMap Server Call failed. Make sure vectormap_server is running. No filtering performed.

@andoh104 andoh104 changed the title Add vector map server2 rm Add vector map server to RuntimeManager Jul 23, 2017
@dejanpan
Copy link

@kitsukawa what is the plan with this PR?

@andoh104 can you re-format the PR description according to https://github.com/CPFL/Autoware/blob/master/docs/pull-request-template.md? thx

@andoh104
Copy link
Author

@dejanpan
I edited the description.
Thanks,

Copy link

@dejanpan dejanpan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kitsukawa kitsukawa merged commit 03430fe into autowarefoundation:develop Aug 28, 2017
@andoh104 andoh104 deleted the AddVectorMapServer2RM branch September 1, 2017 12:14
@yk-fujii yk-fujii mentioned this pull request Sep 21, 2017
21 tasks
@mitsudome-r mitsudome-r added the version:autoware-ai Autoware.AI label Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants