-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pcds-5.9.1 #346
pcds-5.9.1 #346
Conversation
Confirmed that the annoying warning is gone when running locally |
The various failures aside from py312 are for a security CVE that doesn't apply to the as-built env. |
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.
Seems un-objectionable to me 👍
@@ -122,6 +122,8 @@ bluesky-base=1.10.0 | |||
databroker=1.2.5 | |||
# ipython changes make scientists upset and can break experiments | |||
ipython=8.4.0 | |||
# newer line_profiler requires newer ipython | |||
line_profiler=4.1.1 |
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.
Really? I guess they haven't set up their conda feedstocks to be separate.. pip installation has a line_profiler[ipython]
option, but conda doesn't seem to...
ew
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.
For documentation purposes:
line_profiler 4.1.3
has:
constraints :
- ipython >=8.14.0
which I think is what run_constrained looks like when you do conda search --info
4.1.1
has:
dependencies:
- ipython >=0.13
4.1.2
did not make it onto conda
@@ -878,4 +884,4 @@ dependencies: | |||
- python-vxi11==0.9 | |||
- slac-alarm-manager==1.3.0 | |||
- tc-release==0.2.6 | |||
prefix: /cds/home/z/zlentz/miniconda3/envs/pcds-5.9.0 | |||
prefix: /home/runner/miniconda/envs/pcds-5.9.1 |
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.
I feel like I see this line change every time haha
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.
haha, yeah it depends on whether I build the env myself or let the runner do it
It doesn't actually affect the installation of the env from this file
Small release, less ambitious than #345
I grabbed the incr env (py39) from GHA after it passed, then made that build the current-env.
See https://jira.slac.stanford.edu/browse/ECS-6281
I think the degraded torch versions from from xopt's dependencies
Added the Following Packages
PCDS Package Updates
Packages With Degraded Versions
Added the Following Dependencies