Skip to content

Conversation

olia110
Copy link
Contributor

@olia110 olia110 commented Aug 25, 2025

This Pull request:

Changes or fixes:

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

This PR fixes #

Copy link
Contributor

@sanjibansg sanjibansg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just one small comment

@@ -6,6 +6,8 @@
#include "TMVA/RModel.hxx"

#include <sstream>
#include <vector>
#include <string>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we need to include the vector and <string> libraries here?

Copy link

Test Results

    21 files      21 suites   3d 12h 14m 15s ⏱️
 3 561 tests  3 422 ✅  0 💤 139 ❌
72 974 runs  72 813 ✅ 22 💤 139 ❌

For more details on these failures, see this check.

Results for commit d22aab5.

@olia110 olia110 force-pushed the feature/tanh-improvement branch from d22aab5 to 5e39de9 Compare August 26, 2025 09:36
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.

4 participants