Closed joelvbernier closed 5 years ago
psavery
commented
5 years ago @joelvbernier Yeah, I was able to reproduce this crash a couple of weeks ago on a mac I have access to. It's possible that openblas might be causing the crash (note thread 5 is the crash thread with libopenblas). Or maybe it has to do with us running it in a background thread in PySide2.
It's tricky. Maybe if we come up with a conda recipe, the libraries will be more stable with each other. Not sure.
cjh1
commented
5 years ago I am working on a conda recipe, but not sure it will help with this.
joelvbernier
commented
5 years ago I have noticed that condo is stuck on numpy 1.16, while pip is up to 1.17…. But I had issues with the conda install of PySide2. Certainly more tasing needs to be done. I’m at Los Alamos Nat’l Lab for a data analytics workshop, and I am trying to drum up excitement for the software…. Should’ve brought my shit linux laptop :-/
On Aug 27, 2019, at 10:59 AM, Patrick Avery notifications@github.com wrote:
@joelvbernier https://github.com/joelvbernier Yeah, I was able to reproduce this crash a couple of weeks ago on a mac I have access to. It's possible that openblas might be causing the crash (note thread 5 is the crash thread with libopenblas). Or maybe it has to do with us running it in a background thread in PySide2.
It's tricky. Maybe if we come up with a conda recipe, the libraries will be more stable with each other. Not sure.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cryos/hexrdgui/issues/158?email_source=notifications&email_token=AAIZYUQRYJEJPQP75WDW7IDQGVMQDA5CNFSM4IQGWRZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5INPSY#issuecomment-525391819, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIZYURZO4R56DVJOMLDMITQGVMQDANCNFSM4IQGWRZQ.
joelvbernier
commented
5 years ago Last time I checked, I got the GUI to run using PySide2 in the conda-forge channel (not in the defaults), but it still barfed.
On Aug 27, 2019, at 11:01 AM, Chris Harris notifications@github.com wrote:
I am working on a conda recipe, but not sure it will help with this.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cryos/hexrdgui/issues/158?email_source=notifications&email_token=AAIZYUXMVJ6OQYKH52OHL6TQGVMX5A5CNFSM4IQGWRZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5INWDA#issuecomment-525392652, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIZYUR74SNF4ATVPTMIOU3QGVMX5ANCNFSM4IQGWRZQ.
cjh1
commented
5 years ago I have borrowed a 10.14.6 from a colleague so will do some experimenting here.
joelvbernier
commented
5 years ago Excellent — has anyone tried windows (gasp…)? Just curious...
On Aug 27, 2019, at 11:17 AM, Chris Harris notifications@github.com wrote:
I have borrowed a 10.14.6 from a colleague so will do some experimenting here.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cryos/hexrdgui/issues/158?email_source=notifications&email_token=AAIZYUXDRJIHLE2ACBJ56T3QGVOQZA5CNFSM4IQGWRZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5IPEPQ#issuecomment-525398590, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIZYUTDGZ32FWJ3XPNWH7TQGVOQZANCNFSM4IQGWRZQ.
psavery
commented
5 years ago @joelvbernier I have a Windows laptop at home, and I can try it on there later today.
joelvbernier
commented
5 years ago May need to crib some of the Windows-specific build tricks from the old v0.3.x branch conda recipe.
cjh1
commented
5 years ago @joelvbernier I installed hexrdgui on my test mac with pip and see lots of segfaults as you report. I then created a conda environment on the mac using a combination of the conda recipe I have been working on and pip, I messed with the spin boxes for translation a whole bunch and couldn't get it to crash. I have explicity exported the conda environment here. If you download that file and then do the following you should be able recreate my environment:
conda create --name hexrd-test --file spec-file.txt
conda activate hexrd-test
pip install -e <path to hexrdgui repo>Let me know if this help. If is crashes let me know how.
joelvbernier
commented
5 years ago Thanks, will give this a shot.
psavery
commented
5 years ago @joelvbernier I gave it a quick try on Windows this morning. It was (perhaps surprisingly) relatively straightforward. I was able to install it using the same install instructions (using a Visual Studio 2017 command prompt). I tried running a few things, and of the things I tried, everything seemed to be working! I didn't run into any crashes either when modifying the spin box values.
You can see an image attached.
cjh1
commented
5 years ago @joelvbernier We now have a working conda recipe, if you would like to try it out, here is the PR:
https://github.com/cryos/hexrdgui/pull/165
Switch to the branch and then run:
conda build -c cjh1 -c conda-forge conda.recipe/
conda install -c cjh1 -c conda-forge --use-local hexrdguiClosing this now as the conda package seems to have resolved it. We can reopen if we run into more problems.
There were first the troubling segfaults seemingly coming from PySide2, but I have a new one:
Bus error: 10got this after playing with the interactors in the polar mode (tth ranges).
Also, the segfaulting behavior is still there, rendering the GUI essentially unusable on mac OS. Perhaps we need to find build versions of Qy and PySide2 that behave better on the Mac?
Again, this happens without fail when playing with the same interactors long enough, though not always at the same time (sometimes sooner, sometimes later). Here is the stack dump from the most recent segfault: