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

latch=true publisher in rosjs #35

Closed
rctoris opened this issue Apr 15, 2013 · 6 comments
Closed

latch=true publisher in rosjs #35

rctoris opened this issue Apr 15, 2013 · 6 comments

Comments

@rctoris
Copy link
Contributor

rctoris commented Apr 15, 2013

Migrated from https://github.com/RobotWebTools/rosjs/issues/31

@jihoonl :
Can rosjs set Publisher(latch=True) ?

@rctoris
Copy link
Contributor Author

rctoris commented Apr 15, 2013

I think the rosbridge protocol itself would have to be modified to support this.

@odestcj
Copy link
Member

odestcj commented Apr 15, 2013

Can you elaborate? I thought this was an issue with the current implementation of rosbridge_server, not v2 of the protocol?

@rctoris
Copy link
Contributor Author

rctoris commented Apr 15, 2013

Other direction -- to create a latched topic, the client has to specify it somewhere in the protocol. rosbridge has to be responsible for this since a client is never notified when someone subscribes to their topic.

@RainCT
Copy link

RainCT commented Jan 23, 2014

Hi jihoonl,

Not sure why you're referencing issue #1. Just to clarify, my (already merged) patch fixes latched subscribers, it has nothing to do with publishers.

@jihoonl
Copy link
Member

jihoonl commented Jan 24, 2014

Yeah I have just noticed that after referencing the pull request. I will just keep the reference to remind that latch subscription is now supported and only remaining thing is supporting latch publisher.

@jihoonl
Copy link
Member

jihoonl commented Mar 27, 2014

Resolved by the pull requests

@jihoonl jihoonl closed this as completed Mar 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants