MattEqualsCoder / MSUScripter

Application for creating MSUs and related files
MIT License
5 stars 2 forks source link

Not detecting PyMusicLooper #11

Closed codemann8 closed 11 months ago

codemann8 commented 12 months ago

I've installed PyMusicLooper and can run it via py -m pymusiclooper on CLI but this program doesn't detect it

MattEqualsCoder commented 12 months ago

Are you able to run it standalone in the CLI via just typing in pymusiclooper? That's how the MSU Scripter currently executes it rather than executing it via a Python command.

codemann8 commented 12 months ago

I've never heard of being able to execute python programs as you would like an exe. How do you make an installed python program do that?

codemann8 commented 12 months ago

I installed it via their second install option: pip install pymusiclooper and upon running pip show pymusiclooper it gives the dir where it supposedly installs it, but I see no exe files there, so I can't say adding that dir to the PATH variable would help any.

MattEqualsCoder commented 12 months ago

I'm not sure how exactly it's setup to do that. I just followed the pymusiclooper pipx installation instructions on the creator's GitHub and it just worked from what I recall.

I can also try to look into adding an option to specify an execution command to run it in the case where running it in a standalone form doesn't work. I just can't easily do that because python could be in different locations and not in the path environment variable.

codemann8 commented 12 months ago

Could you not just execute it as py -m pymusiclooper? Or have it cacasde attempts, try the current way first, then this, then if fail show the error.

MattEqualsCoder commented 12 months ago

I installed it via their second install option: pip install pymusiclooper and upon running pip show pymusiclooper it gives the dir where it supposedly installs it, but I see no exe files there, so I can't say adding that dir to the PATH variable would help any.

Ah, maybe the difference is I installed via pipx as that's what was recommended by the creator.

As far as running it via py -m, that requires the python directory to be in the path environment variable, which isn't always the case. Perhaps I can try to do some sort of cascade where I try it one way, then fall back to the other way.

Unfortunately I'll be out of town this weekend, so I won't be able to look at this until next week at the earliest.

codemann8 commented 12 months ago

Ok, I see now, your way also requires the Python/Scripts folder from your current users' AppData to be in the PATH variable, so really it's a special case either way

MattEqualsCoder commented 12 months ago

Well, being able to execute py/python anywhere in the CLI requires the python execution directory to be environment path. Usually this happens automatically, but I've had cases where I had to do it manually for whatever reason.

But yeah, I'll give this a look when I get the chance.

MattEqualsCoder commented 11 months ago

@codemann8 This should be fixed in the new release: https://github.com/MattEqualsCoder/MSUScripter/releases/tag/v1.1.1

Let me know if you still run into problems.