ch3njust1n / smpl

Simultaneous Multi-Party Learning Framework
0 stars 0 forks source link

KeyError: 'parameters' #42

Closed ch3njust1n closed 6 years ago

ch3njust1n commented 6 years ago
Traceback (most recent call last):
  File "/home/ubuntu/anaconda2/envs/smpl/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
    self.run()
  File "/home/ubuntu/anaconda2/envs/smpl/lib/python2.7/multiprocessing/process.py", line 114, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ubuntu/smpl/distributed/parameter_server.py", line 356, in __train
    nn.update_parameters(sess['parameters'])
KeyError: 'parameters'
ch3njust1n commented 6 years ago

ok was set to False in ps.__synchronize_parameters() and the peer was notified that it had the best parameters, so it retrieved its best parameters. But then was setting ok=ok and self.cache.set(sess_id, ujson.dumps(sess)), but was short-circuiting because in if-block to retrieve best, wasn't setting ok to True. Stupid ok expressions need to go. Second time this has caused unexpected errors.