Skip to content

tidy: add sanitizers to likelihood fit and minor tidy#931

Merged
KSkwarczynski merged 1 commit into
developfrom
kswkarczynski/tidy/sanitizers
Jun 17, 2026
Merged

tidy: add sanitizers to likelihood fit and minor tidy#931
KSkwarczynski merged 1 commit into
developfrom
kswkarczynski/tidy/sanitizers

Conversation

@KSkwarczynski

Copy link
Copy Markdown
Member

Pull request description

Add following sanitizers:

  • Throw error if ROOT uses Open MP for minuit
  • Don't use adaptive and PCA

Otherwise remove annoying iterator of vector which is quite long and doesn't help much...


@KSkwarczynski KSkwarczynski self-assigned this Jun 11, 2026
@github-actions github-actions Bot added the MCMC 🎲 MCMC related label Jun 11, 2026
@github-actions

Copy link
Copy Markdown

Hi @KSkwarczynski, thank you for contributing to MaCh3! Please wait for MaCh3 developers to review your PR. If no one answers within a week, please message people from this list: https://github.com/orgs/mach3-software/teams/mach3admin . While waiting, please enjoy this

meme

Use this action on your projects. Use jokes on issues instead.

@github-actions

Copy link
Copy Markdown

Hi, I am MaCh3-Telemetry bot

Memory and CPU Usage Summary

### Memory and CPU Usage Summary

| Metric  |     Mean    |      Max    |
|---------|-------------|-------------|
| **RAM** |   357.70 MB |   368.61 MB |
| **CPU** |    97.33 %  |   118.00 %  |

Memory Usage Plot

You can view the memory usage plot directly in the job summary from the link above.

@KSkwarczynski KSkwarczynski force-pushed the kswkarczynski/tidy/sanitizers branch from 78e4943 to 2071d01 Compare June 11, 2026 13:13
@KSkwarczynski KSkwarczynski merged commit 54fd23a into develop Jun 17, 2026
28 checks passed
@KSkwarczynski KSkwarczynski deleted the kswkarczynski/tidy/sanitizers branch June 17, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant