make profiling rate defaults to 101 hz

[stefanosiano]

📜 Description

1. Deprecated profilingTracesIntervalMillis option
2. Added internal profilingTracesHz option
3. Default profilingTracesHz is now 101, to avoid lockstep sampling

💡 Motivation and Context

1. profilingTracesIntervalMillis should not be exposed to the user, at least for now
2. We make reasoning in terms of profiling traces rate, so i added the profilingTracesHz option
3. In order to make sure we avoid lockstep sampling i set it to 101 by default

💚 How did you test it?

Unit test to check profilingTracesIntervalMillis is ignored and profilingTracesHz is used, instead.

📝 Checklist

• I reviewed the submitted code
• I added tests to verify the changes
• I updated the docs if needed
• No breaking changes

🔮 Next steps

[codecov-commenter]

Codecov Report

Merging #2211 (0b265f5) into main (ba925d3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2211   +/-   ##
=========================================
  Coverage     80.65%   80.65%           
  Complexity     3355     3355           
=========================================
  Files           240      240           
  Lines         12324    12324           
  Branches       1633     1633           
=========================================
  Hits           9940     9940           
  Misses         1778     1778           
  Partials        606      606           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[adinauer]

Ah didn't notice the changelog was wrong on main

[stefanosiano]

yeah, probably could even be when i merged another pr 😅. Not sure, though