Moved BasicTimeProfiler and HandlersTimeProfiler to core

[KickItLikeShika]

Fixes #2126

Moved time_profilers from contrib to core.

Check list:

• New tests are added (if a new feature is added)
• New doc strings: description and/or example code are in RST format
• Documentation is updated (if required)

[sdesrozis]

@KickItLikeShika thank you ! Did you take care of keeping the git history of the file time_profilers.py ? (I mean using git mv or mv command ?)

[KickItLikeShika]

@sdesrozis thanks for asking! Yes I did, I always follow this approach https://stackoverflow.com/questions/1043388/record-file-copy-operation-with-git/46484848#46484848, as you can see in the 3 first commits

[vfdev-5]

@KickItLikeShika thank you ! Did you take care of keeping the git history of the file time_profilers.py ? (I mean using git mv or mv command ?)

@sdesrozis this is a good question. I think we didn't do that for previous classes neither.

[KickItLikeShika]

@vfdev-5 I did the same thing while moving LRFinder to core #1998 (comment)

[vfdev-5]

@vfdev-5 I did the same thing while moving LRFinder to core #1998 (comment)

Maybe, but there is no git history for the file : https://github.com/pytorch/ignite/commits/3ef85e1b7af58cfbcf2033d62085a759aa722cc1/ignite/handlers/lr_finder.py . It appears as a new file.

[KickItLikeShika]

@vfdev-5 We can try git mv directly but the file in the contrib will have no history, what do you think?

[vfdev-5]

@vfdev-5 We can try git mv directly but the file in the contrib will have no history, what do you think?

@sdesrozis thoughts ? I think we can continue as it is right now as other files already do not have any history.

[sdesrozis]

We have the full git history, so there is no issue here. Let's continue as we did before.

[vfdev-5]

LGTM