Add fix for checking is_time_series() property based on data_type attr

[tomvothecoder]

Description

• Closes [Bug]: CDAT migration: AttributeError: 'CoreParameter' object has no attribute 'test_start_yr' #877

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules

If applicable:

• New and existing unit tests pass with my changes (locally and CI/CD build)
• I have added tests that prove my fix is effective or that my feature works
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)

[tomvothecoder]

A more explicit version of the main code below:

e3sm_diags/e3sm_diags/driver/utils/dataset.py

Lines 188 to 195 in 4cda0a8

	def is_timeseries(self):
	"""
	Return True if this dataset is for timeseries data.
	"""
	if self.ref:
	return getattr(self.parameters, "ref_timeseries_input", False)
	else:
	return getattr(self.parameters, "test_timeseries_input", False)

[tomvothecoder]

@chengzhuzhang This PR should fix the attribute error (tested with your run script). You can rebase your eamxx_1024 branch on the latest cdat-migration-fy24

EDIT: Actually let me address a few more of the open issues in #871 first so you don't have to keep rebasing.

[chengzhuzhang]

@tomvothecoder thank you. I will keep modifying eamxx_1024, the changes are mostly contained within variable derivation, so it is easy to rebase.