fix: copy trial_columns when copying EventDataFrame

[SiQube]

resolves #1127

the described d.py now returns:

(venv) siqube@c-cube:/tmp/pymovements
$ python d.py 
['n']
['n']

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (c4ed6bc) to head (9195c16).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #1128   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           87        87           
  Lines         3818      3818           
  Branches       679       679           
=========================================
  Hits          3818      3818           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[dkrako]

Thanks for taking care! Can you add a test to prevent regressions?

[dkrako]

As a side note, hotfix: is no valid conventional commit prefix. the correct prefix is fix:.

https://www.conventionalcommits.org/en/v1.0.0/

[dkrako]

This could also be changed in this PR (but can be also treated separately):

pymovements/src/pymovements/gaze/gaze_dataframe.py

Lines 1215 to 1228 in 028bdf6

	def clone(self) -> GazeDataFrame:
	"""Return a copy of the GazeDataFrame.
	Returns
	-------
	GazeDataFrame
	A copy of the GazeDataFrame.
	"""
	gaze = GazeDataFrame(
	data=self.frame.clone(),
	experiment=deepcopy(self.experiment),
	)
	gaze.n_components = self.n_components
	return gaze

It also doesn't clone self.events.

Strange that we have Gaze.clone() and Events.copy().