get_min_max_timestamp functions about ismn HOT 4 CLOSED

Hi Marco, there has been a change in format for the header files (an extra line-break in the second row seems to be the problem). I would suggest we add a line to your code and then include it in the next release.

def get_min_max_timestamp_header_values(file_path):
    with io.open(file_path, mode='r', newline=None) as fid:
        lines = fid.readlines()
        if not lines[1].isspace():
	    first = lines[1]
        else:
	    first = lines[2]
        last = lines[-1]

    min_date = datetime.strptime(first[:16], '%Y/%m/%d %H:%M')
    max_date = datetime.strptime(last[:16], '%Y/%m/%d %H:%M')

    return min_date, max_date

Thanks again and please share more suggestions & improvements, when you find the time!

Cheers, Daniel

from ismn.

Hi Marco, you are right, get_min_max_timestamp_header_values is not working properly. However, if I apply your changes the tests no longer pass. We'll have to look into this, and get back to you. thanks!

from ismn.

@daberer we can close this issue, right?

from ismn.

@wpreimes seems to be fixed in #27, so I'll just close this 👍🏻

from ismn.