AuScope / seed-vault

Other
2 stars 0 forks source link

force uppercase in station filters #132

Closed filefolder closed 1 week ago

filefolder commented 1 week ago

the NETWORK, STATION, LOCATION, CHANNEL parameters are case sensitive but should always be UPPERCASE. it probably makes sense to force these to be .upper() prior to the get_stations call in the event a user does not know this or something strange happens elsewhere

e.g. line 826 in seismoloader.py

    kwargs = {
        'network': net.upper(),
        'station': sta.upper(),
        'location': loc.upper(),
        'channel': cha.upper(),
        'starttime': starttime,
        'endtime': endtime,
        'includerestricted': settings.station.include_restricted,
        'level': settings.station.level.value
    }

and line 630

        kwargs = {
            'network':request[0].upper(),
            'station':request[1].upper(),
            'location':request[2].upper(),
            'channel':request[3].upper(),
            'starttime':UTCDateTime(request[4]),
            'endtime':UTCDateTime(request[5])
        }
filefolder commented 1 week ago

applied change