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

Listener problem back with 0.10.3 #392

Open
trittsv opened this issue Mar 27, 2023 · 3 comments
Open

Listener problem back with 0.10.3 #392

trittsv opened this issue Mar 27, 2023 · 3 comments

Comments

@trittsv
Copy link
Contributor

trittsv commented Mar 27, 2023

Hey, with 0.10.3 the listener not connected problem is back.
In my manually patched 0.10.2 with ad500aa it does not occur.

Just to make sure if i downloaded something wrong,
@tez1szh can you check if it happens again with 0.10.3 in your app too?

@zijian-teng
Copy link
Contributor

Hi @trittsv I checkout cyclone@master, and [email protected], make, make install, and re-complile my app.
I ran the test script 500+ rounds, no event missing observed. (without the fix, event missed every ~10 rounds)

something came into my mind:

  • listener was set in the constructor of the entity, instead of setting the listener after entity was constructed
  • 0.10.3 was installed properly and the header files was updated and complied with in your app

@trittsv
Copy link
Contributor Author

trittsv commented Mar 28, 2023

@tez1szh can you repeat your test with [email protected] + [email protected]?

When i try to compile the cyclone@master with [email protected] then i have a compile error, how did you done it?

TopicTraits.hpp:204:9: error: ‘ddsi_sertype_cdr_data_t’ was not declared in this scope; did you mean ‘ddsi_sertype_derive_t’?

@zijian-teng
Copy link
Contributor

@trittsv Sorry for the confusion, one of my repo was not up-to-date.
I fetch the latest changes and use [email protected](63b6eab), [email protected](0c78734)

I ran the test for ~500 rounds, issue not repreducible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants