ESCOMP / ESCOMP/CTSM

Refactor towersite and neon methods so they are shorter, more managable, and unit-testable

Open
#2,413 1 comment 1 reaction 0 assignees View on GitHub
code health enhancement
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
6d 6h
Merged PRs (30d)
8

Description

We've been working on refactoring NEON python scripts to they are part of the python structure and can tested under there. Many of the methods are too long, this makes them hard to debug, to test, lint-check, and to verify they are correct, as well as making them harder to manage. As these scripts are being worked on we will refactor them so they are shorter.

And we should also remove the disabling of the pylint subroutine length issue.

`# pylint: disable=too-many-statements`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.