Making MacOS unit tests parallel on GH Actions

john-science commented 3 months ago

What is the change?

Making MacOS unit tests parallel on GH Actions.
Again, our Python 3.7 tests are breaking for some numpy reason, so we're dropping 3.7. (Not worth our time to fix / support.)

This reduces the GitHub run time for our macos tests from 14.5 minutes to 4 minutes!

Why is the change being made?

I recently found an update to runLog.py that would fix various test parallelization issues. I am using it here to allow our MacOS unit tests on GitHub Actions to run in parallel.

Checklist

[X] This PR has only one purpose or idea.
[X] Tests have been added/updated to verify any new/changed code.

[X] The code style follows good practices.
[X] The commit message(s) follow good practices.

[X] The release notes have been updated if necessary.
[X] The documentation is still up-to-date in the doc folder.
[X] The dependencies are still up-to-date in pyproject.toml.

albeanth commented 3 months ago

If we're dropping 3.7, the docs need to be updated too https://github.com/terrapower/armi/blob/189075860cf2db6936c93181ce06fc5427520176/doc/user/user_install.rst?plain=1#L15

john-science commented 3 months ago

If we're dropping 3.7, the docs need to be updated too

Well, I was only planning on dropping CI for Python 3.7. You should I should drop support as well?

Thoughts?

albeanth commented 3 months ago

Hmmmm. Well, if we aren't testing for it in CI, then we aren't really maintaining it. And I don't think our docs should start off with using a version of Python that we don't test for. At least that's my thinking. What do you think?

We could update the docs to say install with Py 3.9+ but also have a footnote saying that 3.7 may work, it's just no longer supported.

john-science commented 3 months ago

We could update the docs to say install with Py 3.9+ but also have a footnote saying that 3.7 may work, it's just no longer supported.

Well, I guess we're doing this.

terrapower / armi