GEUS-Glaciology-and-Climate/pypromice

Removing msg_lat and msg_lon from the files?

BaptisteVandecrux opened this issue · 2 comments

Now it is not used anymore because of too low quality.

See why they are not used here:
#152

I'm planning to remove msg_lat and msg_lon from variable.csv as a first fix

At a later point, we should also remove it from:

def getConfig(config_file, inpath, default_columns: Sequence[str] = ('msg_lat', 'msg_lon')):

conf[s]["columns"].extend(default_columns)

try:
ds_list.append(self.readL0file(target))
except pd.errors.ParserError as e:
# ParserError: Too many columns specified: expected 40 and found 38
logger.info(f'-----> No msg_lat or msg_lon for {k}')
for item in ['msg_lat', 'msg_lon']:
target['columns'].remove(item) # Also removes from self.config
ds_list.append(self.readL0file(target))

# append lat and lon to end of message
l0.msg = l0.msg + ',{},{}'.format(lat,lon)

and potentially other places

fixed with #229