windcrusader / tidepredict Goto Github PK

Currently output is always in metres. It would be good to output data in feet for those folks in the USA.

At present the tidegraph generates a graph for a length depending on the constant "GRAPHSPAN". This should rather be the default but if the user gives a time interval as per the textual output then the graph should be for that interval.

Some sensible error checks should take place also.

(tidepredict) λ python tidepredict -l Lyttelton
Refreshing stations list from online source.
ftp request:RETR /uhslc/rqds/pacific/pacific.lst
ftp request:RETR /uhslc/rqds/indian/indian.lst
ftp request:RETR /uhslc/rqds/atlantic/atlantic.lst
Traceback (most recent call last):
File "tidepredict_main_.py", line 66, in process_args
stations = pd.read_csv(constants.STATIONFILE)
File "C:\Users\Jerry\New_Python_Projects\tidepredict\lib\site-packages\pandas\io\parsers.py", line 686, in read_csv
return _read(filepath_or_buffer, kwds)
File "C:\Users\Jerry\New_Python_Projects\tidepredict\lib\site-packages\pandas\io\parsers.py", line 452, in _read
parser = TextFileReader(fp_or_buf, **kwds)
File "C:\Users\Jerry\New_Python_Projects\tidepredict\lib\site-packages\pandas\io\parsers.py", line 936, in init
self._make_engine(self.engine)
File "C:\Users\Jerry\New_Python_Projects\tidepredict\lib\site-packages\pandas\io\parsers.py", line 1168, in _make_engine
self._engine = CParserWrapper(self.f, **self.options)
File "C:\Users\Jerry\New_Python_Projects\tidepredict\lib\site-packages\pandas\io\parsers.py", line 1998, in init
self._reader = parsers.TextReader(src, **kwds)
File "pandas_libs\parsers.pyx", line 382, in pandas._libs.parsers.TextReader.cinit
File "pandas_libs\parsers.pyx", line 674, in pandas._libs.parsers.TextReader._setup_parser_source
FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\Jerry\.tidepredict\stations.csv'

Need to check if I'm searching for information pre 2000 and change the prefix search.

At the moment, I have implemented the skeleton of command line processing using the Python "argparse" package. The idea is to eventually implement most of the options in xtide, but at the moment not all these are implemented.

I think I should remove the command line options not currently supported to make the tool more user friendly.

$ python tidepredict -harmgen -l "La Jolla"
ftp request:RETR /uhslc/rqds/pacific/hourly/h554a.zip
Opening data file:i554a15.dat
Traceback (most recent call last):
File "/home/dlarue/miniconda3/envs/envPyTides/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/dlarue/miniconda3/envs/envPyTides/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "tidepredict/main.py", line 183, in
process_args(args)
File "tidepredict/main.py", line 99, in process_args
loc_code = loc_code)
File "/home/dlarue/tidepredict/tidepredict/processdata.py", line 179, in process_unhw_data
if dateraw[8] == "1":
IndexError: string index out of range

It would be nice to be able to specify a flexible resolution for the graph output. E.g something like -rh 800 -rw 600 for res height and res width of the generated PNG.

I need to write a method to read the station QA doc online when grabbing the station data and in particular make sure I get the local time zone for presenting the results. Other useful information should also be gleaned from this file.

This will affect the format of the saved file for harmonics constants.

At the moment the tool handles timezones in a mostly ok manner, just not that generically. Ideally, in the WeeWx tradition, all time processing should be done in UTC, and conversion to local time handled at the last second before output processing.

What needs to happen is that there should be a new "pre/post-processor" module that converts the passed time into UTC before passing to any tide generation function. The module could then convert to local as a final step when required.

It should be able to:

1. convert a single date time to/from local time from/to UTC
2. convert a list of date times to/from local time from/to UTC