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

Error: invalid wire type 4 at offset 2332 #148

Closed
erksch opened this issue Jun 1, 2020 · 5 comments
Closed

Error: invalid wire type 4 at offset 2332 #148

erksch opened this issue Jun 1, 2020 · 5 comments
Assignees
Labels
Priority: 2 - High 😰 Should be fixed as quickly as possible, ideally within the current or following sprint Severity: 2 - High 😰 Does not cause a failure, but causes the system to be incorrect, incomplete, or inconsistent Status: Investigating 🔍 Something we are looking in to, but not currently in active development Type: Bug 🐛 Some functionality not working in the codebase as intended

Comments

@erksch
Copy link

erksch commented Jun 1, 2020

Description

I tried to run syft.js together with a hosted FL plan for MNIST that I created and hosted with the notebooks from the PySyft repo (/examples/experimental/FL Training Plan). I followed the instructions in the syft.js repo and opened the with-grid example. However when starting the worker, there seem to be a number of issues:

  1. The upload speed test takes extremely long and blocks the whole machine. I had to manually kill my chrome process to exit the syft.js tab. This is weird because the 64MB buffer you created should not take too long to send to a localhost server. I solved this by just returning Promise.resolve(1000) in the getUploadSpeed function.
  2. Next, unserializing the plan protobuf that is returned by the server fails with the following error:
Error: invalid wire type 4 at offset 2332

The unserialization of the model protobuf that is executed right before that works.

How to Reproduce

  1. Setup PyGrid and PySyft
  2. Create and host a plan the PySyft notebook example
  3. Setup syft.js
  4. Click "Start FL Worker"

System Information

  • OS: Ubuntu
  • OS Version: 20.04
  • Language Version: Python 3.7, Node 10.14.2
  • Package Manager Version: pip 19.3. NPM 6.5
@erksch erksch added the Type: Bug 🐛 Some functionality not working in the codebase as intended label Jun 1, 2020
@cereallarceny cereallarceny added Priority: 2 - High 😰 Should be fixed as quickly as possible, ideally within the current or following sprint Severity: 2 - High 😰 Does not cause a failure, but causes the system to be incorrect, incomplete, or inconsistent Status: Investigating 🔍 Something we are looking in to, but not currently in active development Type: Bug 🐛 Some functionality not working in the codebase as intended and removed Type: Bug 🐛 Some functionality not working in the codebase as intended labels Jun 5, 2020
@cereallarceny
Copy link
Member

We will look into this @erksch, sorry for the delay. @vvmnnnkv have you come across this?

@vvmnnnkv
Copy link
Member

vvmnnnkv commented Jun 7, 2020

I've separated upload speed test problem in #150
@erksch could you put more details there?
Thanks!

@vvmnnnkv
Copy link
Member

vvmnnnkv commented Jun 7, 2020

Error: invalid wire type 4 at offset 2332

Should be fixed by #151

@cereallarceny
Copy link
Member

Closed by #151

@erksch
Copy link
Author

erksch commented Jun 8, 2020

Awesome! I'll give it a try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 2 - High 😰 Should be fixed as quickly as possible, ideally within the current or following sprint Severity: 2 - High 😰 Does not cause a failure, but causes the system to be incorrect, incomplete, or inconsistent Status: Investigating 🔍 Something we are looking in to, but not currently in active development Type: Bug 🐛 Some functionality not working in the codebase as intended
Projects
None yet
Development

No branches or pull requests

3 participants