search

AMF-FLX / AMF-BASE-QAQC

AmeriFlux BASE Flux/Met Data QA/QC and Processing
Other
5 stars 7 forks source link

Synchronize AMF-BASE-QAQC utils with general purpose utils functionality #16

Open dschristianson opened 1 year ago

dschristianson commented 1 year ago

Describe the issue that the proposed feature is addressing. Synchronize utilities in this repo with those that are similar to the general purpose utilities used in other parts of the AMP infrastructure. Various utils are grouped in ~/processing/utils.py module in this repo, and there are more utilities specific to AMF-BASE-QAQC than currently exist in the general set. The general set of utilities is a private repo (and will be for near future) so we cannot make it a requirement for AMF-BASE-QAQC.

Describe the solution you'd like. Isolate the utilities in this repo that are similar to those in the general set (i.e., create a new utilities module to contain just the shared ones):

Keep utilities specific to AMF-BASE-QAQC in ~/processing/utils.py.

Describe alternatives you've considered if any Make the general utilities repo public (future project). Mix utilities together in existing ~/processing/utils.py module (Not using this option b/c difficult to keep synchronized and there will be utilities specific to AMF-BASE-QAQC repo that will still need a home).

dschristianson commented 1 year ago

See PR 399 in original repo for some work on updating utils.