-
Notifications
You must be signed in to change notification settings - Fork 8
Conversation
For testing, I really want a dedicated virtual environment. Avoid requirements conflicts that way. BUT, there is no PyQt5 conda package that matches requirements (yet): ``` PackagesNotFoundError: The following packages are not available from current channels: - pyqt5[version='>=5.8'] Current channels: - https://conda.anaconda.org/lightsource2-tag/linux-64 - https://conda.anaconda.org/lightsource2-tag/noarch - https://conda.anaconda.org/prjemian/linux-64 - https://conda.anaconda.org/prjemian/noarch - https://repo.anaconda.com/pkgs/main/linux-64 - https://repo.anaconda.com/pkgs/main/noarch - https://repo.anaconda.com/pkgs/free/linux-64 - https://repo.anaconda.com/pkgs/free/noarch - https://repo.anaconda.com/pkgs/r/linux-64 - https://repo.anaconda.com/pkgs/r/noarch - https://conda.anaconda.org/conda-forge/linux-64 - https://conda.anaconda.org/conda-forge/noarch - https://conda.anaconda.org/aps-anl-tag/linux-64 - https://conda.anaconda.org/aps-anl-tag/noarch - https://conda.anaconda.org/aps-anl-dev/linux-64 - https://conda.anaconda.org/aps-anl-dev/noarch ``` Can requirement be relaxed to fit what is [available](https://anaconda.org/search?q=pyqt5)? Looks as if [pyqt5 5.6](https://anaconda.org/inso/pyqt5) is latest for linux-x86_64. PyPI has [5.13](https://pypi.org/project/PyQt5/) today.
@prjemian, I think the name of the package on anaconda.org is EDIT: corrected the link to refer to the |
Agreed. Going with this conda env setup for now:
which shows (in agreement with https://anaconda.org/search?q=pyqt)
then:
which downloads and builds lots of packages. We'll see if pip tries to update the conda-installed pyqt on my RHEL system today. Yesterday, on my linuxmint system, pip tried to update pyqt which led to version conflicts and the demo would not run. Should not have tried that within my standard bluesky environment.
|
Finally:
tl;dr works for me |
Does this PR need to be updated, then, to deal with the discrepancy in how pip and conda spell PyQt5? |
P.S. Thanks, both for this PR and for taking |
Still need to test this on system with only default channels specified. Might need to add conda-forge. |
No, that was my typo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @prjemian! I suggested a minor change, which will make the steps logically grouped.
bluesky jsonschema matplotlib ophyd pyqt \ | ||
pyzmq qtpy suitcase-jsonl tornado traitlets \ | ||
-c lightsource2-tag | ||
conda activate bluesky_browser |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these 2 conda commands be moved before these lines?
git clone https://github.com/NSLS-II/bluesky-browser
cd bluesky-browser
That will make the consequent installation steps to be in a more logical order.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Absolutely since they no longer rely on requirements.txt
@mrakitin, @danielballan approved. Can this be merged? |
Thanks!
…On Tue, Aug 27, 2019, 6:57 AM Maksim Rakitin ***@***.***> wrote:
Merged #49 <#49> into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#49?email_source=notifications&email_token=AARMUMFLLYJRNHZTKGNY553QGUJBHA5CNFSM4ILYDZNKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTIXYPAI#event-2586806145>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AARMUMAYU7IYKHJHH335GRDQGUJBHANCNFSM4ILYDZNA>
.
|
For testing, I really want a dedicated virtual environment. Avoid requirements conflicts that way. BUT, there is no PyQt5 conda package that matches requirements (yet):
Can requirement be relaxed to fit what is available? Looks as if pyqt5 5.6 is latest for linux-x86_64. PyPI has 5.13 today.