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.

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

Update to latest Miniforge (and use Conda) #598

Merged
merged 2 commits into from Apr 16, 2024
Merged

Conversation

jakirkham
Copy link
Member

@jakirkham jakirkham commented Apr 2, 2024

Updates to the latest Miniforge 24.3.0-0 in the Docker image used by Heroku to build and deploy the webservice.

Xrefs:

Checklist

  • Pushed the branch to main repo
  • CI passed on the branch

@jakirkham
Copy link
Member Author

Marking as draft as we may want to jump to 24.3, which looks ready ( conda-forge/miniforge#574 )

@jakirkham jakirkham mentioned this pull request Apr 2, 2024
2 tasks
@jakirkham
Copy link
Member Author

Updated to 24.3.0-0. Marking as ready for review

@jakirkham jakirkham changed the title [WIP] Update to latest Miniforge (and use Conda) Update to latest Miniforge (and use Conda) Apr 16, 2024
@jakirkham jakirkham marked this pull request as ready for review April 16, 2024 21:37
@jakirkham jakirkham requested a review from beckermr April 16, 2024 21:38
@beckermr
Copy link
Member

Can you remind me why we think this won't crash the heroku server due to memory usage again?

I forget what changed in the latest conda.

@beckermr
Copy link
Member

ahhhh nvm: #585 (comment)

@beckermr
Copy link
Member

let's try it!

@beckermr beckermr merged commit 521bb01 into main Apr 16, 2024
3 checks passed
@beckermr beckermr deleted the use_latest_miniforge branch April 16, 2024 21:55
@jakirkham
Copy link
Member Author

Thanks Matt! 🙏

Yep Conda 24.3.0 should have the memory fix we need 👍

Looks like it built successfully! 🎉

@jakirkham
Copy link
Member Author

FWIW have run a few re-renders successfully. So think this is working well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants