CABLE-LSM / benchcab

Tool for evaluation of CABLE land surface model
https://benchcab.readthedocs.io/en/latest/
Apache License 2.0
2 stars 3 forks source link

Prioritize system binaries over paths set by user #252

Closed abhaasgoyal closed 7 months ago

abhaasgoyal commented 7 months ago

Resolves #220

Merge Message

analysis3-unstable had a package named ld whose library search paths were different from system's ld (/usr/bin/ld). This commit ensures that during compilation process, any binaries would be searched in system paths first before conda paths.

Tested with CABLE commit 67a52dc5721f0da78ee7d61798c0e8a804dcaaeb Environment: Cloned analysis3-unstable and remove benchcab.

codecov[bot] commented 7 months ago

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (e3f1f9e) 59.65% compared to head (7c8fa5a) 59.61%. Report is 9 commits behind head on main.

:exclamation: Current head 7c8fa5a differs from pull request most recent head 6706a73. Consider uploading reports for the commit 6706a73 to get more accurate results

Files Patch % Lines
benchcab/benchcab.py 50.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #252 +/- ## ========================================== - Coverage 59.65% 59.61% -0.04% ========================================== Files 30 30 Lines 2191 2199 +8 ========================================== + Hits 1307 1311 +4 - Misses 884 888 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.