I have added the code corresponding screenshots for the boxplot testing function in test_datetime.py file. and completed the portion for Axes.boxplot of issue#26864"
this is the screenshot of boxplot by given code..

-->

PR checklist

"addresses part of[TST]: increase unit test coverage #26864 " is in the body of the PR description toBoxplot for datetime
new and changed code istested
Plotting related features are demonstrated in anexample
New Features andAPI Changes are noted with adirective and release note
Documentation complies withgeneral anddocstring guidelines

boxplot code updated!!

5cc4668

melissawm added topic: testing topic: date handling labels

Oct 9, 2023

ksunden requested changes

Oct 9, 2023

View reviewed changes

Copy link

Member

ksunden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

We'd prefer to use the axes interface over theplt methods for this test.

Also I'd like to see another axes that shows avert=False boxplot to make sure that the unit handling works for horizontal boxplots.

Could also argue forpositions to be given as datetimes (in another plot...) to make sure that it handles that as well.

lib/matplotlib/tests/test_datetime.py

Comment on lines +97 to +98

		datums = np.array([datetime.datetime.toordinal(d) for d in data])
		plt.boxplot(datums, labels=['my data'], showfliers=True)

Copy link

Member

ksundenOct 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Suggested change

	datums=np.array([datetime.datetime.toordinal(d)fordindata])
	plt.boxplot(datums,labels=['my data'],showfliers=True)
	plt.boxplot(data,labels=['my data'],showfliers=True)

We actually very specificallywant the datetimes passed into the plotting methods here. as it stands, this test is not actually testing the unit conversion at all.