search

ericmandel / pyds9

Python connection to SAOimage DS9 via XPA
76 stars 41 forks source link

Wrong executable name for DS9 in OSX #105

Open npirzkal opened 2 years ago

npirzkal commented 2 years ago

pyds9 looks for /Applications/SAO ImageDS9.app when running in OSX but the current versions of DS9 are now installed as /Applications/SAOImageDS9.app (no space). This causes pyds9 to complain that it cannot find any executable and it does not start ds9 automatically. Removing the space on line 75 of pyds9.py fixed it for me. This is using SAOImageDS9 v. 8.2.1

ericmandel commented 2 years ago

The version of pyds9.py in github looks for SAOImageDS9.app as well as the older filename with spaces. Can you please try upgrading to the guthub version?

On Thu, Jun 2, 2022 at 10:55 AM npirzkal @.***> wrote:

pyds9 looks for /Applications/SAO ImageDS9.app when running in OSX but the current versions of DS9 are now installed as /Applications/SAOImageDS9.app (no space). This causes pyds9 to complain that it cannot find any executable and it does not start ds9 automatically. Removing the space on line 75 of pyds9.py fixed it for me. This is using SAOImageDS9 v. 8.2.1

— Reply to this email directly, view it on GitHub https://github.com/ericmandel/pyds9/issues/105, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXDPC5LG644YRW74HQJVNTVNDDO5ANCNFSM5XVIC45A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

npirzkal commented 2 years ago

Thanks. I just did a pip install so I assumed it was not patched :-)

N

On Jun 2, 2022, at 11:28 AM, Eric Mandel @.***> wrote:

The version of pyds9.py in github looks for SAOImageDS9.app as well as the older filename with spaces. Can you please try upgrading to the guthub version?

On Thu, Jun 2, 2022 at 10:55 AM npirzkal @.***> wrote:

pyds9 looks for /Applications/SAO ImageDS9.app when running in OSX but the current versions of DS9 are now installed as /Applications/SAOImageDS9.app (no space). This causes pyds9 to complain that it cannot find any executable and it does not start ds9 automatically. Removing the space on line 75 of pyds9.py fixed it for me. This is using SAOImageDS9 v. 8.2.1

— Reply to this email directly, view it on GitHub https://github.com/ericmandel/pyds9/issues/105, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXDPC5LG644YRW74HQJVNTVNDDO5ANCNFSM5XVIC45A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/ericmandel/pyds9/issues/105#issuecomment-1144998076, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZWJHGUJDZQVMBGKRJ2ZY3VNDHJRANCNFSM5XVIC45A. You are receiving this because you authored the thread.

ericmandel commented 2 years ago

Agreed that it would be great to have pip updated, but we have no python expert to do that sort of work.

Sent from my iPhone

On Jun 2, 2022, at 11:30 AM, npirzkal @.***> wrote:

 Thanks. I just did a pip install so I assumed it was not patched :-)

N

On Jun 2, 2022, at 11:28 AM, Eric Mandel @.***> wrote:

The version of pyds9.py in github looks for SAOImageDS9.app as well as the older filename with spaces. Can you please try upgrading to the guthub version?

On Thu, Jun 2, 2022 at 10:55 AM npirzkal @.***> wrote:

pyds9 looks for /Applications/SAO ImageDS9.app when running in OSX but the current versions of DS9 are now installed as /Applications/SAOImageDS9.app (no space). This causes pyds9 to complain that it cannot find any executable and it does not start ds9 automatically. Removing the space on line 75 of pyds9.py fixed it for me. This is using SAOImageDS9 v. 8.2.1

— Reply to this email directly, view it on GitHub https://github.com/ericmandel/pyds9/issues/105, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXDPC5LG644YRW74HQJVNTVNDDO5ANCNFSM5XVIC45A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/ericmandel/pyds9/issues/105#issuecomment-1144998076, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZWJHGUJDZQVMBGKRJ2ZY3VNDHJRANCNFSM5XVIC45A. You are receiving this because you authored the thread.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

npirzkal commented 2 years ago

The way I do it is pretty straight forward:

python setup.py sdist bdist_wheel twine upload dist/*

You will need twine installed but I do this with a simple "pip install twine" You can register and create an account at https://pypi.org/project/twine/

I hope that helps.

ericmandel commented 2 years ago

Thanks, if I get time in the next month, I'll look into it ... but unfortunately, this really is a question of support that can't be fixed ad hoc.