Reduce duplication 2

[CBroz1]

Description

This PR builds on #1050 after confirmation from @khl02007 that the variable name change of amplitude_thresh -> amplitude_thresh_uV was not due to underlying packages. This moves the following funcs to a utils file...

• _check_artifact_thresholds
• _compute_artifact_chunk
• _init_artifact_worker

This also reduces the line count for both clusterless and spiked versions of get_decoding_data_for_epoch in decoding.v0

I was also able to move the following to utils...

• git_interval_range - from decoding.v1
• the majority of get_firing_rate save for the piece that depends on another class method (a good candidate for round 3)

Do the following need to be public functions? If not, I think they can be removed, as they are not called within the codebase

• spikesorting_merge.get_spike_indicator
• spikesorting.vX.recording._get_recording_timestamps
• decoding.v1.X.get_firing_rate
• Relatedly: linearization.vX.TrackGraph.get_networkx_track_graph: this is a relatively simple func that, if it doesn't need to be public, can probably be merged into calling functions

Checklist:

• No. This PR should be accompanied by a release: (yes/no/unsure)
• N/a. If release, I have updated the CITATION.cff
• No. This PR makes edits to table definitions: (yes/no)
• N/a. If table edits, I have included an alter snippet for release notes.
• N/a. If this PR makes changes to position, I ran the relevant tests locally.
• I have updated the CHANGELOG.md with PR number and description.
• N/a. I have added/edited docs/notebooks to reflect the changes

[edeno]

I would leave this as it is useful to get the array.

[CBroz1]

Sure - I'll remove the comment. Do you feel the same about the others listed in the description?

[edeno]

Yeah, I would leave them in for now.