Implement temporal_extent extraction of .csv

o2r-project / geoextent

Python library for extrating geospatial extent of files and directories with multiple data formats

https://o2r.info/geoextent/

MIT License

1 stars 4 forks source link

Implement temporal_extent extraction of .csv #20

Closed YouQam closed 4 years ago

nuest commented 4 years ago

Currently supported column names: time, timestamp

[ ] Support other names:
- date
- datetime
[ ] normalise column names before checking (to lower case, remove delimiters) and add test for different cases, e.g. Time, TimeStamp, TIME, Date etc.
[ ] log a warning if more than one column was found (e.g. date AND time), then only use one of them

YouQam commented 4 years ago

There is a bug in getting time interval due to including headers.

Screenshot from 2020-02-28 00-45-06

YouQam commented 4 years ago

When more than one column was found, it takes the first one. As in the test case: https://github.com/YouQam/geoextent/commit/1a4934ee75c3eba3790794addafc2911825272b7

nuest commented 4 years ago

@YouQam For the test in https://github.com/YouQam/geoextent/commit/1a4934ee75c3eba3790794addafc2911825272b7 to actually test something, wouldn't it be good if the DATE column had different values, so you are sure the TIME column is chosen?

YouQam commented 4 years ago

@YouQam For the test in YouQam@1a4934e to actually test something, wouldn't it be good if the DATE column had different values, so you are sure the TIME column is chosen?

Sure about it and I will change values to be more sure.

YouQam commented 4 years ago

dataset has been changed. https://github.com/YouQam/geoextent/commit/7f021d34b52ccc419f4bfcb62a63a8ad9dc67de2