Refactor setter for performance and readability

[mfisher87]

Originally posted by @weiji14 in #550 (comment):

The setter is here:

icepyx/icepyx/quest/dataset_scripts/argo.py

Lines 79 to 85 in a77b685

	@params.setter
	def params(self, value):
	"""
	Validate the input list of parameters.
	"""
	self._params = list(set(self._validate_parameters(value)))

Suggestions:

1. The code is calling list(set()) twice (once in the setter, and once in _validate_parameters), we could at least remove one of those.
2. The _valid_params list could be turned into a set for faster membership tests, see https://docs.astral.sh/ruff/rules/literal-membership.