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

Bunch of fixes #269

Conversation

Hadatko
Copy link
Member

@Hadatko Hadatko commented May 9, 2022

Pull request

Choose Correct

  • bug
  • feature

Describe the pull request

tcp python server is using event for waiting for sock connection as it was eating cpu.
formatting python code
tests are build before running
c specific includes made c specific

To Reproduce

working on python tests which were showing these errors

Expected behavior

much less cpu usage
ability to run python test

Screenshots

Desktop (please complete the following information):

  • OS:
  • eRPC Version:

Steps you didn't forgot to do

  • I checked if other PR isn't solving this issue.
  • I read Contribution details and did appropriate actions.
  • PR code is tested.
  • PR code is formatted.

Additional context

@Hadatko Hadatko self-assigned this May 9, 2022
tcp python server is using event for waiting for sock connection as it was eating cpu.
formatting python code
c specific includes made c specific

Signed-off-by: Cervenka Dusan <cervenka@acrios.com>
@Hadatko Hadatko force-pushed the feature/fix_server_cpu_usage branch from a3b88e5 to 01c8f37 Compare May 9, 2022 09:59
@Hadatko
Copy link
Member Author

Hadatko commented May 9, 2022

Hi @MichalPrincNXP , this should be ready for merge. There is one fix for TCP python transport and two fixes for Rpmsg python transport.

@Hadatko Hadatko added this to the 1.10.0 milestone May 9, 2022
@MichalPrincNXP MichalPrincNXP merged commit 82a8222 into EmbeddedRPC:develop Jun 6, 2022
@MichalPrincNXP
Copy link
Member

Thank you!

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

Successfully merging this pull request may close these issues.

None yet

2 participants