mnnpy icon indicating copy to clipboard operation
mnnpy copied to clipboard

comput_angle=True generates error

Open chlee-tabin opened this issue 6 years ago • 1 comments

Hi,

I wanted to obtain the angles and the following error is produced:

multiprocessing.pool.RemoteTraceback: 
"""
Traceback (most recent call last):
  File "/n/app/python/3.6.0/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/n/app/python/3.6.0/lib/python3.6/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
TypeError: find_shared_subspace() got multiple values for argument 'mat1'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "20180706_mnnCorrect_withangle.py", line 307, in <module>
    "lib3",
  File "/home/cl266/scanpy_environment/lib/python3.6/site-packages/mnnpy/mnn.py", line 126, in mnn_correct
    svd_mode=svd_mode, do_concatenate=do_concatenate, **kwargs)
  File "/home/cl266/scanpy_environment/lib/python3.6/site-packages/mnnpy/mnn.py", line 192, in mnn_correct
    angle_out = p_n.map(find_subspace_job, correction_in)
  File "/n/app/python/3.6.0/lib/python3.6/multiprocessing/pool.py", line 260, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/n/app/python/3.6.0/lib/python3.6/multiprocessing/pool.py", line 608, in get
    raise self._value
TypeError: find_shared_subspace() got multiple values for argument 'mat1'

This occurs even in case where I specified n_jobs = 1.

The following environment and version has been used for mnnpy:

$ pip show mnnpy
Name: mnnpy
Version: 0.1.9.4
Summary: Mutual nearest neighbors correction in python.
Home-page: http://github.com/chriscainx/mnnpy
Author: Chris Kang
Author-email: [email protected]
License: BSD 3
Location: /home/cl266/scanpy_environment/lib/python3.6/site-packages
Requires: numba, pandas, numpy, anndata, scipy
Required-by: 

chlee-tabin avatar Jul 07 '18 03:07 chlee-tabin

Will fix it soon

chriscainx avatar Jul 10 '18 06:07 chriscainx