Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installing flirpy inside conda, split_seqs command is not recognized #27

Closed
GZuin opened this issue Aug 25, 2020 · 3 comments
Closed

Installing flirpy inside conda, split_seqs command is not recognized #27

GZuin opened this issue Aug 25, 2020 · 3 comments

Comments

@GZuin
Copy link

GZuin commented Aug 25, 2020

Attempted to install flirpy inside of a conda environment on both Windows 10 and Ubuntu 18.04. In both cases, after installing through pip, no command split_seqs is found.

I can safely import flirpy inside of python, but after reading the split_seq example, I'm not sure what to do to split a seq file into images through python.

@jveitchmichaelis
Copy link
Contributor

jveitchmichaelis commented Aug 25, 2020

This seems to be an issue with installing via pip. Possibly poetry related? If you install using python setup.py install, it should place the script on your path (I just tested this in a new conda env on my Ubuntu machine). For now you can just copy the code in scripts and it'll work.

On Windows this is a pain because of the way that Windows interprets files. It will find it, but won't execute properly (this is on the todo list), so you should also just copy the code and run it from somewhere.

@jveitchmichaelis
Copy link
Contributor

jveitchmichaelis commented Aug 25, 2020

So this is a poetry issue...

python-poetry/poetry#241
python-poetry/poetry-core#40

It seems like there is a lot of confusion in poetry over what a script should be - whether it's some "build" script or a user script that gets installed along with a package. Support has now been added, but right now it seems like there is no obvious way to do this until that PR gets closed.

@GZuin GZuin closed this as completed Aug 27, 2020
@jveitchmichaelis
Copy link
Contributor

@GZuin I need to push a new version shortly anyway, so I'll do it through Twine just as a precaution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants