fixing small issues in copy_instrument and magproc.py

See merge request !330

Small but important change to only load metadata that has the status of reviewed and not rejected,new, etc.

Variometer api additions

See merge request !327

add metadata variable to unpacking

See merge request !326

Adding ignore statement to pyproject.toml for pip safety issue

See merge request !325

Pull inputs in run(), not run_as_update()

See merge request !324

Fixed the bugs cuasing interval to be defined as none. This required changes...

See merge request !323

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.

This simply returns True from can_produce_data(), which is appropriate
because 1) a stateful algorithm should be able to procude data as long
as it starts from a valid state, adn 2) SqDistAlgorithm itself checks
this state. If we ever implement a different stateful algorithm, it
should do something similar.

For years, when invoking geomag-algorithms' update mechanism, inputs
were pulled and checked from the run_as_update() method inside the
Controller class. I never understood this. It seemed to break the
logic of the update mechanism if no inputs were available for the
current inteval/gap. This might even be the source of noted issues
where running back-filling scripts didn't behave as expected, and
those scripts needed to be run multiple times.

What's more, the run_as_update() method logically seems like the
most appropriate place to read *outputs* and check for gaps. Whereas
the run() method seems like the most appropriate place to read
inputs, apply algorithms, and write out outputs.

In any case, this change should not break any existing code. It
should only allow the update mechanism to complete, every time, and
as originally intended, rather than be short-circuited when input
data are missing for the current interval (but might be available
for previous intervals)..

New flag metadata class

See merge request !292

New Instrument class and Observatory class edits

See merge request !290

Added metadata Dict to parse_params and additional functionality to get_metadata to handle metadata Dict after response.