Skip to content

Conversation

@H0R5E
Copy link
Contributor

@H0R5E H0R5E commented Nov 13, 2023

This PR is to exclude R2020b and R2021a from the Windows test runners while they seem to be broken.

Specific changes are:

  • Merged windows and linux jobs, using the matrix to vary version differences
  • Excluded R2020b and R2021a from Windows jobs
  • Include latest in the Linux jobs

EDIT: Opened bug report in setup-matlab #76.

@H0R5E H0R5E requested a review from akeeste November 13, 2023 16:10
@H0R5E H0R5E added the Tests/CI related WEC-Sim tests or Continuous Integration label Nov 13, 2023
Copy link
Contributor

@akeeste akeeste left a comment

Choose a reason for hiding this comment

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

Approved. Thanks @H0R5E

@akeeste akeeste self-assigned this Nov 13, 2023
@H0R5E
Copy link
Contributor Author

H0R5E commented Nov 13, 2023

@akeeste, not sure that using the caches are worth it. If you have a look here, the storage used goes well over the 10GB limit with just 3 versions of MATLAB cached. Shall I remove the option before this is merged?

@akeeste
Copy link
Contributor

akeeste commented Nov 13, 2023

@H0R5E what is being cached by setup-matlab? Is it saving a specific MATLAB installation for a future test?

@H0R5E
Copy link
Contributor Author

H0R5E commented Nov 13, 2023

@H0R5E what is being cached by setup-matlab? Is it saving a specific MATLAB installation for a future test?

Yeah, that's the idea. It would reduce the installation time on subsequent runs. The caches are saved per repository, but the images are 4Gb-ish each, which means only 2 versions can be realistically cached, whereas we test about 10.

@H0R5E
Copy link
Contributor Author

H0R5E commented Nov 13, 2023

@akeeste, you can see the same effect on the WEC-Sim GH Actions cache here after running this PR.

@akeeste
Copy link
Contributor

akeeste commented Nov 13, 2023

@H0R5E thanks for confirming. I agree; I don't see MATLAB caching being very useful right now, given their size.

@H0R5E
Copy link
Contributor Author

H0R5E commented Nov 13, 2023

OK, I've removed the caching. Mathworks are looking at the issue with the broken versions already in #76.

@H0R5E H0R5E merged commit 63181c8 into WEC-Sim:dev Nov 13, 2023
@H0R5E H0R5E deleted the exclude_windows_versions branch November 13, 2023 17:25
jtgrasb added a commit that referenced this pull request Dec 1, 2023
* Update gh-pages-redirect.html

Added meta description to the website SEO
Meta descriptions may be included in search results to concisely summarize page content.

* Remove direct github installations (#1149)

* Exclude R2020b and R2021a from Windows test runners (#1166)

This commit excludes R2020b and R2021a from the Windows test runners.

Specific changes are:
+ Merged windows and linux jobs, using the matrix to vary version differences
+ Excluded R2020b and R2021a from Windows jobs
+ Include latest in the Linux jobs

* updating release notes doc

* fix typo

* updated most wecsim version

---------

Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>
Co-authored-by: Mathew Topper <damm_horse@yahoo.co.uk>
Co-authored-by: jtgrasb <jtgrasb@sandia.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tests/CI related WEC-Sim tests or Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants