chore: use M_PI instead of PI parameter

[Divyansh-jain2]

Description of Change

Checklist

• Added description of change
• PR title follows semantic commit guidelines
• Search previous suggestions before making a new one, as yours may be a duplicate.
• I acknowledge that all my contributions will be made under the project's license.

Notes:

[realstealthninja]

Thank you!

[realstealthninja]

have you checked all the tests?

[realstealthninja]

might need to add this before cmath

#define _USE_MATH_DEFINES
#include <cmath>

[realstealthninja]

@Divyansh-jain2 there is just a single conversation to resolve to get this merged!