[critical path] Add tolerance for negative one weight due to precision issues, improvements

[briancoutinho]

What does this PR do?

• Add tolerance for negative one weight due to precision issues

• No need to have overlaid director exist, now just mkdir it

  Before submitting

• [ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

  • [x] N/A

• [x] Did you write any new necessary tests?

  • [ ] N/A

• [ ] Did you make sure to update the docs?

  • [x] N/A

• [ ] Did you update the changelog?

  • [x] N/A

[facebook-github-bot]

@briancoutinho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@briancoutinho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 90.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 90.83%. Comparing base (9cad1f2) to head (abff786).

Files	Patch %	Lines
hta/analyzers/critical_path_analysis.py	80.00%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #138 +/- ## ========================================== + Coverage 90.80% 90.83% +0.02% ========================================== Files 52 52 Lines 5186 5192 +6 ========================================== + Hits 4709 4716 +7 + Misses 477 476 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[facebook-github-bot]

@briancoutinho merged this pull request in facebookresearch/HolisticTraceAnalysis@9ef98987b3dbb6efdc8ce8a40b99e663acee7edb.