MismatchedSeriesLength Datacheck

[MichaelFu512]

Pull Request Description

Closes #4297
Adds mismatched series length datacheck

Visual of what this PR does

Say there was a multiseries dataset with 5 unique series_id that each show up 20 times (so there are 100 entries in the dataset).

Let's say the first row was dropped, meaning that series_id "0" only has 19 entries while the rest of the series_id have 20 entries.

Using the mismatched series length datacheck, the validate function will return:

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

[codecov]

Codecov Report

Patch coverage: 100.0% and project coverage change: +0.1% 🎉

Comparison is base (93e7b97) 99.7% compared to head (4422f0e) 99.7%.

Additional details and impacted files

@@           Coverage Diff           @@
##            main   #4296     +/-   ##
=======================================
+ Coverage   99.7%   99.7%   +0.1%     
=======================================
  Files        355     357      +2     
  Lines      39511   39577     +66     
=======================================
+ Hits       39391   39457     +66     
  Misses       120     120             

Files Changed	Coverage Δ
evalml/data_checks/__init__.py	100.0% <100.0%> (ø)
evalml/data_checks/data_check_message_code.py	100.0% <100.0%> (ø)
...data_checks/mismatched_series_length_data_check.py	100.0% <100.0%> (ø)
..._tests/test_mismatched_series_length_data_check.py	100.0% <100.0%> (ø)

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

[eccabay]

Just a few nitpicks, but nothing blocking!

[jeremyliweishih]

LGTM - just some nit suggestions. I would also recommend adding adding a test case where there are multiple majority lengths i.e two modes for the majority length) and note the behavior considering multiple modes in the docstring!

[jeremyliweishih]

Suggested change

	# get the majority length
	majority_len = statistics.mode(series_id_len.values())

[MichaelFu512]

Is this faster than what I currently have or is it more concise (or both)?

[jeremyliweishih]

Its more concise but I believe its the same time complexity. You can double check here:

• https://github.com/python/cpython/blob/3.11/Lib/statistics.py#L686
• https://github.com/python/cpython/blob/3.11/Lib/collections/__init__.py#L624

[MichaelFu512]

I think I'll stick to what I have currently since what I have lets me easily exit out early if all the series have the same length, which is something that Becca asked to do in one of her comments.