Remove instrument_sensitivity if it exist. If it doesn't exist, then
an instrument_polynomial must describe the response, which is how the
Geomagnetism Program specifies an offset and gain response. We do not
attempt to remove any frequency-depenent response since these all seem
tailored to seismic data (e.g., no DC/zero-frequency response).

Added necessary check within FDSNSNCL.py to assign the sncl.channel as LF1,LF2,LFZ when X,Y,Z are requested by the user.

REsolved comments made by Josh. The rotation function is now sampling frequency agnostic, while also relying on the original stream created under get_timeseries.

Removed previous lines of code that checked metadata for Azimuth and Dip and replaced it with a new function that rotates the Stream object if channels X, Y, or Z, are requested and returns the appropriate channel after rotation.

Added severl changes to allow for users to retrieve one minute and one hour data from the avaiable one-second data via the FDSN client.

Reviewed comments made by Josh and removed debugging print statements and changed _get_channel_start to get_channel_start within SNCL.py and any other script which references this function.

Fixed the bugs cuasing interval to be defined as none. This required changes to the FDSNFactory.py after fixing this another bug was found in IAGAWriter2002.py, This was also fixed. The change made to the test_metadate.py script was something that Nich told me about that hasnt been merged in yet. Happy to take it out if need be but I left it in for the time being.

Python version >=3.10 don't need to use the typing module types imported
(e.g., `from typing import List, Tuple`), but if we continue to stick with
these, geomag-algorithms should be backward compatible to py3.8. Some day
we'll have to bump up our minimum python version, but not today.

I have reomved what I considered unnecessary functionality from the new FDSN_Factory. This has greatly condensed the length of the factory. I have left a few lines of code commented out because I am not sure if they warrant concern. These pertain to the stdout and stderr variable and comments Payton made regarding how Obspy factories write to stdout rather than stderr.

Committing two new files, FDSNSNCL and FDSNFactory. The factory is modeled after the MiniseedFactory with the additional functions that retrive azimuth and dip. The Factory also relies on the get_waveforms method of obspy instead of get_timeseries. The instrument response is also attached and then that response is altered using the remove_response method in Obspy. This returns the response in nT, but we may have to change this depending on whether there are variometers with a frequency response. The __init__.py file was also edited to incooporate the FDSNSNCL and FDSNFactory.

Made docstring consistent with code wrt snclMapper option.

Allow EdgeFactory to change which mappings will be used from common channels
(i.e., X, Y, Z, etc), to SEED SNCLs (station, network, channel, location)
codes. For now we default to "legacy", for backward compatibility, but we
can specify snclMapper="geomag" to use the modern SNCLs used by the USGS
Geomagnetism Program.

Prior to this, a scale factor of 1000 was applied to all data written
(via raw input client) prior to writing to edge server. This is still
the default, but now user can change this scale factor. This will
allow the EdgeFactory to be used to read data it did not write, and
rescale it if necessary. Note: default is 1000, not 1, for backward
compatibility.

Add obspy.clients.neic to client list for retrieving data from edgecwb
using the CWBQuery protocol (uses miniseed blocks instead of tracebufs).
We will leave earthworm in-place for now, but only if the input port is
2060. 2060 is the only port used for earthworm protocal in any of the
edgecwb nodes used by Geomag, so this should be OK. Eventually we want
to remove earthworm from "Edge"Factory, perhaps creating a separate
factory dedicated to earthworm, although, to be honest, we'll probably
never used earthworm again once we've migrated everthing to miniseed.

These options were never used. Presumably they were once intended to allow
a separate edgecwb host to be specified for non-real time data. It just isn't
obvious why this would be desirable in 2024.

As per discussion in issue #77 with Dave K., this commit adds delta/2
to the requested `endtime` to ensure that starttime and endtime are
fully inclusive in Edge-related I/O factories. I would have liked to
generalized this and reduce the duplicate code between EdgeFactory
and MiniSeedFactory, but couldn't figure out how to do this without
adding considerably *more* code to the project than what is in this
commit.

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

Remove old sncl methods from factories, fix interval code for predefined channels, support chan.loc format