JCSDA-internal / ioda-converters

Various converters for getting obs data in and out of IODA
8 stars 2 forks source link

changing file permissions #1507

Closed BenjaminRuston closed 2 months ago

BenjaminRuston commented 2 months ago

Description

A number of python scripts do not have execute permission, to be consistent they should be changed.

note the process followed to update permissions for a hypothetical converter_file.py below are:

chmod 755 converter_file.py
git update-index --chmod=+x  converter_file.py
git commit -m 'changing file permissions'

Issue(s) addressed

Resolves #1506

Impact

Lateral move, for current implementations.

Checklist

BenjaminRuston commented 2 months ago

@jeromebarre I've noticed these converters in the compo directory that could also be changed.

This doesn't impact anything at present as generally everything is called by preceding the call with python3

here are the files I could change, no changes are made to the contents just an update to the permissions:

aeronet_aod2ioda.py
omps_o3_nm_h52ioda.py
tempo_nc2ioda.py
tropess_co_nc2ioda.py
tropomi_no2_co_nc2ioda.py
viirs_aod2ioda.py
viirs_biaswriter.py
viirs_l1bnc2ioda.py
BenjaminRuston commented 2 months ago

thanks @jeromebarre for the review, did you want me to change the permission on the compo ones as well?

jeromebarre commented 2 months ago

yes please

BenjaminRuston commented 2 months ago

@travissluka any input otherwise think this is now ready