Skip to content

Conversation

loriab
Copy link
Member

@loriab loriab commented May 11, 2023

Description

User API & Changelog headlines

Dev notes & details

  • I was seeing 17m to solve the env last week, so let's switch to mamba solve.
  • Have to turn off analytic Hessian testing for windows since no ints.
  • Looks like it was testing twice -- one exe and one lib. Not really necessary.
  • Turn c-b back on. It was only off so I could get a green check on the release last week.

Checklist

Status

  • Ready for review
  • Ready for merge

@loriab loriab force-pushed the v18_misc_round3 branch from 8b0b6d4 to 7fa1e62 Compare May 11, 2023 05:34
@loriab loriab force-pushed the v18_misc_round3 branch from ede1b93 to da9557f Compare May 18, 2023 07:18
@loriab loriab added this to the Psi4 1.9 milestone May 18, 2023
Copy link
Contributor

@JonathonMisiewicz JonathonMisiewicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remind me what the blocker on Windows high AM integrals is, again?

@loriab
Copy link
Member Author

loriab commented May 24, 2023

Remind me what the blocker on Windows high AM integrals is, again?

No particular high AM blocker, but there is a high derivative blocker for Windows. Basically, it compiles ~1099 files in 3h, then hangs on the final file and fails to build under the 6h. Jet saw this, too, when he was building L2 on his local Windows box. It worked when left overnight, but on Azure through c-f it's a strict 6h limit. No lowering of AM for the Hessian ints helped any.

Since psi has so few analytic Hessians and they add so much expense to the L2 build in general, I figured the most helpful sol'n was to allow psi to live w/o them w/o too much fuss.

@jturney jturney added this pull request to the merge queue Aug 10, 2023
Merged via the queue into psi4:master with commit f83a46b Aug 10, 2023
@loriab loriab deleted the v18_misc_round3 branch August 10, 2023 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants