Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bump ambit, clear addons from azure #2546

Merged
merged 5 commits into from
Apr 13, 2022
Merged

bump ambit, clear addons from azure #2546

merged 5 commits into from
Apr 13, 2022

Conversation

loriab
Copy link
Member

@loriab loriab commented Apr 13, 2022 •
edited

Description

Todos

  • Psi doesn't use ambit directly so never has required a version. Now that it's had its python interface updated (kill the boost!), most uses in the presence of Psi4 will need the new version, so let's not let lingering old versions pass.
  • The ecosystem GHA seems to have settled in nicely to testing addons, so let's remove that responsibility from the Azure CI lanes.
    • leaving dftd3 and gcp since they're fast, easy, and important
    • now addons tested with less compiler variety on Linux, but I don't think that's too big a problem

Status

  • Ready for review
  • Ready for merge

@loriab loriab added this to the Psi4 1.6 milestone Apr 13, 2022
@loriab loriab added the build label Apr 13, 2022
@loriab loriab merged commit 5a56a74 into master Apr 13, 2022
@loriab loriab deleted the loriab-patch-2 branch April 13, 2022 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jturney jturney jturney approved these changes

@zachglick zachglick zachglick approved these changes

@JonathonMisiewicz JonathonMisiewicz JonathonMisiewicz approved these changes

Assignees
No one assigned
Labels
build
Projects
None yet
Milestone
Psi4 1.6
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@loriab @jturney @zachglick @JonathonMisiewicz