dbekaert / RAiDER

Raytracing Atmospheric Delay Estimation for RADAR
Apache License 2.0
70 stars 39 forks source link

Update RC files instead of overwriting them #652

Closed garlic-os closed 2 months ago

garlic-os commented 3 months ago

Description

This changes the behavior RAiDER.models.credentials.check_api to only update the one set of credentials in a .netrc file pertinent to RAiDER's operation instead of overwriting the file.

Motivation and Context

Fixes issue #638 where other credentials in .netrc can be lost.

How Has This Been Tested?

Unit tests were added that excerise check_api for every supported weather model to ensure check_api is behaving correctly, and crucially, that it no longer modifies unrelated .netrc credentials.

Type of change

Checklist:

jlmaurer commented 2 months ago

@jhkennedy do you want to take a quick look at this before I merge?