Umdp3 checker in python #153
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
investigations as to whether it's worked and how it can be improved.
and is to std out whereas the origianl wrote to std err. It's not clear if the Python version terminated after the first file with errors, or genuinely didn't spot the errors in the other files. Branch used to look for errors is : https://code.metoffice.gov.uk/svn/um/main/branches/dev/roddysharp/vn13.9_deliberate_umdp3_failures
… all the edited files.
… in trunk mode...
…p3_checkas I think Perl did it for both cases..
…or a Trunk in suite..
methodology. Plus making the branch processing of files changes return a list.
…clarity and to reduce the size of umdp3.py.
hints as at every step it suggests something else...
…'t commit these changes, which I won't need at the end...
…ecker. Style checkers hold the list of files they check
r-sharp
commented
Dec 18, 2025
The files I added some time ago are no longer there...
yaswant
reviewed
Dec 18, 2025
yaswant
reviewed
Dec 18, 2025
yaswant
reviewed
Dec 18, 2025
r-sharp
commented
Dec 18, 2025
Again, a file I thought I'd deleted prior to creating the pull request.
Some of which seem to be reversing line breaking it imposed last time I ran it. It also now makes more of these files fail flake8 and pylint -//Super//
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Summary
Replacing the old Perl umdp3 compliance checking script with a half baked Python one.
Changes
Remove all old Perl files.
Add Python translations after some heavy refactoring.
This provides a framework, and some very simple tests to run on Fortran code.
It will need some tweaking as we stumble over the defficiencies we see when used in anger.
Impact
Replaces a set of Perl scripts which was hardwired to FCM to a Python version capable of using git_bdiff
Issues addressed
Resolves
One of the last remaining Perl script uses in the toolset
Hardwiring of our style checker to FCM
Checklist