Cain, Payton David authored 3 years ago and Cain, Payton David committed 3 years ago

Update geomagio/DerivedTimeseriesFactory.py, geomagio/TimeseriesFactory.py, geomagio/edge/EdgeFactory.py, geomagio/edge/MiniSeedFactory.py files

Change trailing whitespace from previously edited comment.

Line was too long, moved last word to next line to change line length from 80 character limit

Changes comments in parse_string function from the comments contained in the IAGA2002 file

Change comments inside of parse_string function from the IAGA2002 comments

Change comments in parse_string function from those contained in IAGA2002

Changes the comments for the parse_string function from IAGA2002 file's comments
Please enter the commit message for your changes. Lines starting

Erin (Josh) Rigler authored 6 years ago and Claycomb, Abram Earl committed 6 years ago

- allow adjusted and provisional strings in --type controller argument
- allow adjusted, provisional, and definitive strings in _get_url() in TimeseriesFactory
- return appropriate abbreviation for adjusted/provisional data type
- translate adjuste/prvisional to A0 location code in EdgeFactory
- only write out DECBAS for variation/reported data types
- translate adjusted/provisional to A0 location code in IMFJSONWriter.py
- in several places I set it up so that 'variation' and 'reported' were synonymous,
  although I am not sure I found every such instance.

I'm not sure this is functionally different from what there was previously. Is this what you were suggesting in your comment on this PR @jmfee?

TimeseriesFactory.get_timeseries() pads to the requested interval with NaNs,
filling in start/end gaps. TimeseriesFactory.put_timeseries() is supposed to
do this (according to Jeremy Fee), but it was trimming NaNs from the start/
end before writing to file. This PR fixes this by using ObsPy's Stream.trim()
method, in a similar manner to TimesereisFactory.get_timeseries(), to pad to
the specified start and end of the urlInterval..