Both u7277320 and sheepfan0828 are my personal Github account

Is it okey that some checks are not passed?

The linting would need to be addressed.

However, more fundamentally, this is not exercising the datetime unit machinery that is expected here, and in fact pie in general does not use the units system (and it's unclear what it even would mean).

As such I think pie should just be removed from this file all together (much like we did to table/spectral methods in #27072)

The list from the original issue/stubs was intentionally a bit broad but part of the idea was to evaluate whether it was using units at all. In this case it is not, and so it should just be removed.

@ksunden Sorry, what do you mean by that? I cannot understand, can you please explain it more specific?

This method falls into the category that passing datetimes to it directly does not make sense and therefore should just be removed (From #26864):

iii: If passing datetime / united data to the method does not make any sense or it should not work open a PR removing that test method and similarly reference this issue.

By calling to_ordinal here, you are not passing the datetime objects, and so are not doing what this test file is trying to exercise. This is because the idea of this test file does not make sense for pie, and therefore the xfail stub (the entire method) should just be removed altogether.

This method falls into the category that passing datetimes to it directly does not make sense and therefore should just be removed (From #26864):

iii: If passing datetime / united data to the method does not make any sense or it should not work open a PR removing that test method and similarly reference this issue.

By calling to_ordinal here, you are not passing the datetime objects, and so are not doing what this test file is trying to exercise. This is because the idea of this test file does not make sense for pie, and therefore the xfail stub (the entire method) should just be removed altogether.

So do you mean that is the method that i am testing with (pie this case) is not accepting datetime object so that the testing function for pie should be removed from the file? If so, i need to remove it from test_datetime.py right now and then create another pull request that i have removed the pie. Is that right?

You can update this PR, no need to make a new one, but yes, I am saying that pie should just be removed from this file.

I took the liberty of updating the PR title as we are no longer smoke testing this method.

Thanks @Sheepfan0828 and congratulations on your first Matplotlib PR! We hope to hear from you again.