wingtorres / morphometrix

A PyQt5 Gui for photogrammetry and morphometry
MIT License
17 stars 7 forks source link

Width measuring lines below image #8

Closed CharlotteBahamas2017 closed 1 year ago

CharlotteBahamas2017 commented 2 years ago

I'm using __MorphoMetriX_V3_test_220122__.py on a macbook pro running macOS Monterey 12.1

Once you have loaded an image, if you 'measure' the whale from the bottom of the picture to the top of the picture using 'Measure Lengths', and then hit 'Measure Widths' the measure lines are below the image - it works correctly if you measure from the top to the bottom

Attached an image showing this

Feel free to contact me for any more info.

Charlotte Dunn cdunn@bahamaswhales.org

Screenshot 2022-02-04 at 8 44 35 AM
wingtorres commented 2 years ago

HI @CharlotteBahamas2017 sorry you are having this issue! However, I cannot seem to reproduce it. Are you using the latest version from github?

CharlotteBahamas2017 commented 2 years ago

hi hi

I’m using this version that KC sent me in an email

__MorphoMetriX_V3_test_220122__.py

On 7 Feb 2022, at 12:45 PM, wingtorres @.***> wrote:

HI @CharlotteBahamas2017 https://github.com/CharlotteBahamas2017 sorry you are having this issue! However, I cannot seem to reproduce it. Are you using the latest version from github?

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031742130, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5RF6NH6OOON6IKUCXDU2AAKLANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

hmmm okay can you just try "git clone https://github.com/wingtorres/morphometrix.git" then run morphometrix from within that directory w/ "python -m morphometrix"? You'll need to activate a python environment with the correct dependencies installed beforehand.

CharlotteBahamas2017 commented 2 years ago

hi

that didn’t work

On 7 Feb 2022, at 12:57 PM, wingtorres @.***> wrote:

hmmm okay can you just try "git clone https://github.com/wingtorres/morphometrix.git https://github.com/wingtorres/morphometrix.git" then run morphometrix from within that directory w/ "python -m morphometrix"? You'll need to activate an environment with the correct dependencies installed beforehand.

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031752847, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5S6J5F655GDHWX53H3U2ABYHANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

can you tell me the error message you receive?

CharlotteBahamas2017 commented 2 years ago

hi - sure - I attached an image of it

‘morphometrix’ is a package and cannot be directly executed

On 7 Feb 2022, at 1:24 PM, wingtorres @.***> wrote:

can you tell me the error message you receive?

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031779061, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5X6GG264T4USUV3MF3U2AE6PANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

are you using the -m option? python -m morphometrix works for me.

CharlotteBahamas2017 commented 2 years ago

yep ..

On 7 Feb 2022, at 1:28 PM, wingtorres @.***> wrote:

are you using the -m flag? python -m morphometrix works for me.

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031783045, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5XUCH4UEUBTEAINTPTU2AFNRANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

hmmm okay... Would you show me the output of both conda list and pip list?

CharlotteBahamas2017 commented 2 years ago

sure

conda list:

(qt6) @. ~ % (qt6) @. ~ % conda list

packages in environment at /Users/lotts/miniconda3/envs/qt6:

#

Name Version Build Channel

ca-certificates 2021.10.8 h033912b_0 conda-forge certifi 2021.10.8 py38h50d1736_1 conda-forge collatrix 1.0.8 pypi_0 pypi dbus 1.13.18 h18a8e69_0
easygui 0.98.2 pypi_0 pypi expat 2.4.4 he49afe7_0 conda-forge gettext 0.19.8.1 h1f1d5ed_1 conda-forge glib 2.55.0 0 conda-forge icu 64.2 h6de7cb9_1 conda-forge jpeg 9e h0d85af4_0 conda-forge libblas 3.9.0 13_osx64_openblas conda-forge libcblas 3.9.0 13_osx64_openblas conda-forge libcxx 12.0.0 h2f01273_0
libffi 3.3 hb1e8313_2
libgfortran 5.0.0 9_3_0_h6c81a4c_23 conda-forge libgfortran5 9.3.0 h6c81a4c_23 conda-forge libiconv 1.15 h0b31af3_1006 conda-forge liblapack 3.9.0 13_osx64_openblas conda-forge libopenblas 0.3.18 openmp_h3351f45_0 conda-forge libpng 1.6.37 h7cec526_2 conda-forge llvm-openmp 12.0.1 hda6cdc1_1 conda-forge ncurses 6.3 hca72f7f_2
numpy 1.22.1 py38h9d72dae_0 conda-forge openssl 1.1.1l h0d85af4_0 conda-forge pandas 1.4.0 py38ha53d530_0 conda-forge pcre 8.45 he49afe7_0 conda-forge pillow 9.0.0 pypi_0 pypi pip 21.2.4 py38hecd8cb5_0
pyqt 5.9.2 py38h655552a_2
pyqt6 6.2.2 pypi_0 pypi pyqt6-qt6 6.2.2 pypi_0 pypi pyqt6-sip 13.2.0 pypi_0 pypi pyqt6-webengine 6.2.1 pypi_0 pypi pyqt6-webengine-qt6 6.2.2 pypi_0 pypi python 3.8.12 h88f2d9e_0
python-dateutil 2.8.2 pyhd8ed1ab_0 conda-forge python_abi 3.8 2_cp38 conda-forge pytz 2021.3 pyhd8ed1ab_0 conda-forge qt 5.9.7 h8cf7e54_3 conda-forge readline 8.1.2 hca72f7f_1
scipy 1.7.3 py38hd329d04_0 conda-forge setuptools 58.0.4 py38hecd8cb5_0
sip 4.19.8 py38h0a44026_0
six 1.16.0 pyh6c4a22f_0 conda-forge sqlite 3.37.0 h707629a_0
tk 8.6.11 h7bc2e8c_0
wheel 0.37.1 pyhd3eb1b0_0
xz 5.2.5 h1de35cc_0
zlib 1.2.11 h4dc903c_4
(qt6) @.*** ~ %

On 7 Feb 2022, at 1:34 PM, wingtorres @.***> wrote:

hmmm okay... Would you show me the output of both conda list and pip list?

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031788931, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5TUONBVYGLHVM4PQALU2AGETANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

CharlotteBahamas2017 commented 2 years ago

(qt6) @.*** ~ % pip list Package Version


certifi 2021.10.8 collatrix 1.0.8 easygui 0.98.2 numpy 1.22.1 pandas 1.4.0 Pillow 9.0.0 pip 21.2.4 PyQt6 6.2.2 PyQt6-Qt6 6.2.2 PyQt6-sip 13.2.0 PyQt6-WebEngine 6.2.1 PyQt6-WebEngine-Qt6 6.2.2 python-dateutil 2.8.2 pytz 2021.3 scipy 1.7.3 setuptools 58.0.4 six 1.16.0 wheel 0.37.1 (qt6) @.*** ~ %

On 7 Feb 2022, at 1:34 PM, wingtorres @.***> wrote:

hmmm okay... Would you show me the output of both conda list and pip list?

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031788931, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5TUONBVYGLHVM4PQALU2AGETANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

That all looks good to me. Can you tell me the output of ls from the same location you tried python -m morphometrix? Thanks for your patience - I'm sure we will figure it out.

CharlotteBahamas2017 commented 2 years ago

(qt6) @. ~ % ls Applications @. Creative Cloud Files Creative Cloud Files fig4_rvw2.pdf Creative Cloud Files (archived) (1) foo.eps Desktop gadm36_BHS_0_sp.rds Desktop.png get-pip.py Desktop.res.png iCloud Drive (Archive) Desktop\bigger.clk installs.jsd Desktop\md245a03_1_00_48_841.clk java.log.10950 Documents java.log.11005 Downloads java.log.1260 Dropbox java.log.1387 Google Drive java.log.17039 Library java.log.17863 Miniconda3-py39_4.10.3-MacOSX-x86_64 2.42.40 PM.sh java.log.2281 Movies java.log.2381 Music java.log.2576 MyBiminiShapefile.dbf java.log.2846 MyBiminiShapefile.prj java.log.3630 MyBiminiShapefile.shp java.log.690 MyBiminiShapefile.shx java.log.986 Pictures lib Public marine Raven Pro marmapcoord-82;22;-77;26.5_res_1_anti.csv Raven Pro 1.4.aif matlab_crash_dump.11384-1 Raven Pro 1.6 matlab_crash_dump.11678-1 Relocated Items matlab_crash_dump.187-1 Rplot.jpeg matlab_crash_dump.3213-1 Rplot01.eps matlab_crash_dump.8868-1 Rplot02.pdf matlab_crash_dump.9903-1 Rplot03.png miniconda3 Send Registration morphometrix Sites opt Tresors personal Untitled.pdf.png profiles.bin Untitled.png rosm.cache Zotero socnetv-data (qt6) @.*** ~ %

On 7 Feb 2022, at 1:38 PM, wingtorres @.***> wrote:

That all looks good to me. Can you tell me the output of ls from the same location you tried python -m morphometrix? Thanks for your patience - I'm sure we will figure it out.

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031792223, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5W5VCXJX3T3XNL7T23U2AGSFANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

okay hopefully this will work. Can you cd morphometrix then try running python -m morphometrix from there? I assume the morphometrix directory is what you just cloned from github.

CharlotteBahamas2017 commented 2 years ago

hi

ok that works but same behaviour, measure from right to left and the width lines are not on the whale

On 7 Feb 2022, at 1:46 PM, wingtorres @.***> wrote:

okay hopefully this will work. Can you cd morphometrix then try running python -m morphometrix from there? I assume the morphometrix directory is what you just cloned from github.

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031798461, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5TYSMZSSCED4TLHNBTU2AHPVANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

Okay great! We should be using the same code now. Makes it especially strange that you are still getting the error though... I just tested it with the image you posted initially with the ticket and the widths appear in the correct location.

wingtorres commented 2 years ago

I don't know off the top of my head the best way to debug this. Are the widths consistently shifted downward along the same axis as the length line?

wingtorres commented 2 years ago

would you post four examples starting from right/left and up/down?

CharlotteBahamas2017 commented 2 years ago

do you want photos or a video?

On 7 Feb 2022, at 2:02 PM, wingtorres @.***> wrote:

would you post four examples starting from right/left and up/down?

— Reply to this email directly, view it on GitHub https://github.com/wingtorres/morphometrix/issues/8#issuecomment-1031811595, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJPO5SCRVW3DEPFNPMB2XLU2AJLZANCNFSM5NR2OXYQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.

wingtorres commented 2 years ago

Four of the same photo with lengths drawn from each of the four cardinal directions with corresponding widths would be great. That might help us diagnose what is going on