fix for the pip-installed standardiser exe

[UnixJunkie]

I patched the pip-installed script in this way.
I don't know how to check if this is the proper fix for the pip package.
I just guess this is the way to fix it.

[UnixJunkie]

If this is integrated and the pip package is updated, please let me know
so that I can test.
It is very nice for end users if they can install the software via pip.
Thanks a lot,
Francois.

[UnixJunkie]

Hello,
Did you have a chance to test this correction?
Thanks a lot,
Francois.

[flatkinson]

Francois,     Apologies, again. I am a bit snowed under with unrelated work. Once I've cleared my backlog I will get onto this.         Francis

On 18/01/2019 07:53, Francois Berenger wrote: Hello, Did you have a chance to test this correction? Thanks a lot, Francois. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#12 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFeK00jEqJHqfdQ-Pj5dqkdsNiiJSlYZks5vEX1_gaJpZM4Y04s2>.

-- Dr Francis L Atkinson Chemogenomics Group European Bioinformatics Institute (EMBL-EBI) European Molecular Biology Laboratory Wellcome Genome Campus Hinxton Cambridge CB10 1SD United Kingdom (01223) 494473

[UnixJunkie]

Maybe @mnowotka can confirm this is a proper fix.

[UnixJunkie]

not merged yet?!
I have to fix the install for all my colleagues who want to use this tool...

[simonbray]

Hi, can this be merged please? The pip install is broken without it:

(standardiser) simon@simon-notebook:~$ standardiser -h
Traceback (most recent call last):
  File "/home/simon/miniconda3/envs/standardiser/bin/standardiser", line 5, in <module>
    from standardiser.bin.standardiser import main
ImportError: No module named standardiser

[simonbray]

Suggested change

	'standardiser=standardiser.bin.standardise_mols:main']},
	'standardise-mols=standardiser.bin.standardise_mols:main']},

[simonbray]

if the intent was renaming the command

[UnixJunkie]

@simonbray you can do:

pip3 install chemo-standardizer

This is a fixed version of the same package.
I did this because the maintainers of standardiser were not responsive enough.

[simonbray]

@UnixJunkie thanks for the tip!

I help maintain a conda package for standardiser on conda-forge; this also offers a functional installation of version 0.1.9: conda install standardiser -c conda-forge.

Still it would be nice if this were fixed upstream.