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.

Added the @interval property to store the data interval based on the channel start for Variometers. Hoping this fixes the issues you saw in mage1. I also added an FDSNSNCL_test.py script.

This reverts commit ee6f3c81.

This reverts commit f10f33f3.

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.

Changed the return of the function _get_orientations from tuple[float,float] to Tuple[float,float]. I tried using List but this did not accept multiple arguments, therefore I imported Tuple alongside List from the typing module.

Created a new branch to contain changes to IRISFactory.py and IRISSNCL.py, the channels reported by ASL and the expected channel coversions have been changed so that LF1 and LF2 correspond to V and U. The reported azimuth and dip is also checked and changed if the polarity is reversed.

Revised the minute definition in ObservatoryMetadata and VariometerMetadata. Removed the ObservatoryMetadata variable from IRISFactory and replaced it with the VariometerMetadata variable and an optional variable that points to the VariometerMetadata module.

Updated the IRISFactory to support the new VariometerMetadata class. Also removed unnecessary lines of code in VariometerMetadata.py and ObservatoryMetadata.py.