MetOffice / MetOffice/lfric_core
Extend logger to include application name on start
Nobody has claimed this yet.
- Dominant language
- Fortran
- Stars
- 26
- Forks
- 73
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 4
Description
At the moment we pass the application name around a lot simply in order to send it to log files. There seems to be little value in this and it bloats argument lists.
There only seems to be value in tagging a log file as coming from a particular application once at the start of execution. Every other instance should be removed and argument lists cleaned up where there is no longer any need to pass the application name.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the logger initialization at application startup and every place the application name is passed to logging. Confirm how the name is written to log files, then identify redundant arguments and their call sites. Done means the application is tagged once at startup and unnecessary application-name parameters are removed without changing logging behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100