mattjj / pyhsmm-autoregressive

autoregressive plugin

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Simple fix to get working with Python 3.6 (on MacOS)

jmarkow opened this issue · comments

Maybe I'm doing something funky here, but to get this working appropriately on my machine with Python 3.6 I had to change the line linked here to,

 from autoregressive.messages import resample_arhmm

If I don't make this change I get the following error when resampling a model,

Traceback (most recent call last):                                                                                                                           
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/bin/moseq2-model", line 11, in <module>
    load_entry_point('moseq2-model', 'console_scripts', 'moseq2-model')()
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Users/jmarkow/Development/python/moseq2_model/moseq2_model/cli.py", line 214, in learn_model
    file=sys.stdout)
  File "/Users/jmarkow/Development/python/moseq2_model/moseq2_model/train/util.py", line 19, in train_model
    model.resample_model(num_procs)
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/pyhsmm/models.py", line 442, in resample_model
    self.resample_states(num_procs=num_procs)
  File "/Users/jmarkow/anaconda/envs/moseq2_model_python3/lib/python3.6/site-packages/autoregressive/models.py", line 240, in resample_states
    from messages import resample_arhmm
ModuleNotFoundError: No module named 'messages'

I also tested this change with a Python 2.7 environment and it didn't introduce any additional issues. Seems like a harmless change on my end.

Thanks, @jmarkow -- I pushed a fix in 19899d5