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

Fix for Sh > 2.* issue, and new environment files and template for running on maths2 (Exeter) #255

Merged
merged 7 commits into from
Oct 19, 2023

Conversation

sit23
Copy link
Contributor

@sit23 sit23 commented Jul 25, 2023

As described in issue #247, versions of sh>2.* have been causing the isca python front-end to fail. This is because sh>2.* has changed the default return type of some of its commands, returning str types rather than sh.RunningCommand.

This fix adds a new function to check the type of sh outputs and only asks for .stdout when a sh.RunningCommand type is returned. This should allow the front end to function with versions of sh=1.* and sh=2.*.

Also included here are new environment files and template files for running with intel compilers on Exeter's new maths2 machine. They are included here for my own convenience, but should be of use to others.

… fortran compilers, intel mpi, hdf5, netcdf-c and netcdf-fortran all compiled from scratch.
…f they are sh.RunningCommand types. If they are, then their output is decded as a str, and if not the output is left alone.
dawzeki

This comment was marked as outdated.

Copy link
Contributor

@daw538 daw538 left a comment

Choose a reason for hiding this comment

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

Changes look good to me from a read of the code.

Nice to have the env files for the new maths machines!

@sit23 sit23 merged commit 5b72697 into ExeClim:master Oct 19, 2023
1 check 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.

3 participants