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

Changed Github runner from Conda to Mamba #265

Merged
merged 32 commits into from
Nov 2, 2023

Conversation

ben-l-p
Copy link
Collaborator

@ben-l-p ben-l-p commented Oct 25, 2023

Changed Github runner from Conda to Mamba

The recent change to the new Conda environment involved swapping out minimal environment used by the Github runner to the regular environment. This causes the Ubuntu runner to fail due to running out of its 7GB of RAM, and so new SHARPy versions could not be verified.

To fix this, Conda has been swapped out with Mamba, which is a faster and less memory-intensive drop-in replacement. This should not have any effect on running SHARPy.

Fixed Typos in Linear_Goland_Flutter.ipynb

Small errors in example notebook have been fixed, should not have any effect on the results.

Copy link
Collaborator

@kccwing kccwing left a comment

Choose a reason for hiding this comment

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

Thanks! @ben-l-p

@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2023

Codecov Report

Merging #265 (5a1b2da) into main (b938c5f) will not change coverage.
The diff coverage is n/a.

❗ Current head 5a1b2da differs from pull request most recent head ac2f321. Consider uploading reports for the commit ac2f321 to get more accurate results

@@           Coverage Diff           @@
##             main     #265   +/-   ##
=======================================
  Coverage   67.69%   67.69%           
=======================================
  Files         166      166           
  Lines       26709    26709           
=======================================
  Hits        18081    18081           
  Misses       8628     8628           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ben-l-p ben-l-p changed the title Fixed typos in linear_goland_flutter.ipynb Changed Github runner from Conda to Mamba Nov 1, 2023
@ben-l-p ben-l-p merged commit ee7206f into main Nov 2, 2023
7 checks passed
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.

None yet

3 participants