Warnings vs ValueErrors with user provided params

[VeckoTheGecko]

We have some warnings along the lines of

Parcels/parcels/field.py

Lines 232 to 238 in 5fff42c

	if self.time_periodic is not False and self.allow_time_extrapolation:
	warnings.warn(
	"allow_time_extrapolation and time_periodic cannot be used together. allow_time_extrapolation is set to False",
	FieldSetWarning,
	stacklevel=2,
	)
	self.allow_time_extrapolation = False

Thoughts on just raising a ValueError instead? Its incorrect usage of the API, and I think explicitly failing would be nicer than implicitly working under different settings

[VeckoTheGecko]

I don't think its a major thing either way. Perhaps worth it only for v4

[erikvansebille]

Yes, good point that a ValueError would be more appropriate and cleaner here; the code then also don't need to implicitly decide that time_periodic is a more important setting than allow_time_interpolation