dtcenter / METplus

Python scripting infrastructure for MET tools.
https://metplus.readthedocs.io
Apache License 2.0
94 stars 37 forks source link

Hotfix: Allow symbolic link to run_metplus.py to run #2500

Closed georgemccabe closed 5 months ago

georgemccabe commented 5 months ago

by getting the real path of the script before trying to find the directory above the script to import metplus and produtil

Pull Request Testing

Created sym link to run_metplus.py in another directory and confirmed that it failed using develop (cannot import produtil). Confirmed that it works with the fix.

Confirm that this fix resolved the issue you experienced. Ensure that all automated tests pass

We could consider adding a test that creates a sym link to run_metplus.py and tries to call it from there.

There are diffs from a previous change that will likely flag diffs in this PR, but they can be resolved once the truth data is updated.

Pull Request Checklist

See the METplus Workflow for details.

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 7920728047

Details


Totals Coverage Status
Change from base Build 7837387956: 0.0%
Covered Lines: 8590
Relevant Lines: 9441

💛 - Coveralls