[BUG] Multi version install in the nightly container

[tfeher]

Describe the bug
To install cuml in the nightly container, normally I just run python setup.py install from the cuml/python folder. Something has changed recently, and now the same command in nightly image does a multi version install.

In practice it means that instead of putting the code under /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml, it is installed at something like /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0.16.0a0+814.g461cbf2a7.dirty-py3.7-linux-x86_64.egg/cuml.

What is the correct way to install the python wrappers without any version suffix? I can work around by copying the files manually, but I guess there is a better solution than this:

cp -r /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0.16.0a0+814.g461cbf2a7.dirty-py3.7-linux-x86_64.egg/cuml /opt/conda/envs/rapids/lib/python3.7/site-packages/

Steps/Code to reproduce bug

git clone https://github.com/rapidsai/cuml.git
nvidia-docker run -e HOST_USER_ID=0 -v$PWD:/mydata -w/mydata  --rm -it -p 8888:8888 rapidsai/rapidsai-dev-nightly:0.16-cuda10.1-devel-ubuntu18.04-py3.7 bash

# inside the container
cd cuml/python
python setup.py install

ls -ltrd /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml* 

Expected behavior
Install the library without version suffix.

[wphicks]

@tfeher, you said this was a recent change? As far as I can tell, this is the same behavior in 0.15 and 0.16, but I may be misunderstanding the issue.

Output in 0.15:

(rapids) root@8b29f60efdfc:/rapids/cuml/python# ls -ltrd /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml*
drwxrwxrwx 29 root root 4096 Aug 26 09:08 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml
drwxrwxrwx  2 root root 4096 Aug 26 09:08 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0+untagged.1.g918df35-py3.7.egg-info
drwxr-xr-x  4 root root 4096 Sep 24 16:31 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0+untagged.1.g918df35-py3.7-linux-x86_64.egg

Output in 0.16:

rapids@76d2e7cf1dad:/rapids/cuml/python$ ls -ltrd /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml* 
drwxrwxrwx  2 root   root   4096 Sep 24 09:43 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0+untagged.1.gce14638-py3.7.egg-info
drwxrwxrwx 28 root   root   4096 Sep 24 09:43 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml
drwxr-xr-x  4 rapids rapids 4096 Sep 24 16:33 /opt/conda/envs/rapids/lib/python3.7/site-packages/cuml-0+untagged.1.gce14638-py3.7-linux-x86_64.egg

I'm sure I'm probably missing something here; can you point out what I'm not seeing? Or maybe this was a pre-0.15 change?

[tfeher]

@wphicks thanks for looking into this. I must be wrong while thinking that it was a recent change. I have also checked with older containers and I confirm what you have found: even with 0.14 stable we have the same behavior that I report here. (Also tried checking out branch-0.15 and installing it in 0.15 container, with the same result). So please ignore the "recently" word.

Otherwise, the issue remain the same: I expect that after running python setup.py install, an import cuml statement would import the just installed package, which is not the case.

[wphicks]

Ah great! Thanks for tracking it back even further. I'll remove the "needs triage" label; this is definitely worth looking into further.

[github-actions]

This issue has been marked rotten due to no recent activity in the past 90d. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.

[tfeher]

The issue still exist in the recent dev containers.

[github-actions]

This issue has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.