Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning about long formulas still relevant? #953

Open
nsmith- opened this issue Apr 22, 2024 · 1 comment
Open

Warning about long formulas still relevant? #953

nsmith- opened this issue Apr 22, 2024 · 1 comment

Comments

@nsmith-
Copy link
Collaborator

nsmith- commented Apr 22, 2024

We still have a warning

Executing factory with a string of length  1948  > 1000, could trigger a bug: ...

in the code at

if len(X) > 1000:
print(
"Executing factory with a string of length ",
len(X),
" > 1000, could trigger a bug: ",
X,
)

and around since 2012 25333c4#diff-6aff0212eae895e959674ba1ef93b157076c49d94d89513acbb130b05f72d480R35-R36
Do we know if/when RooFit fixed this?

@guitargeek
Copy link
Contributor

I don't know about this problem. That means it has either been resolved by now, or it was never reported to ROOT. I would bet it got resolved. Maybe try to just remove this check, and if there still is a problem then we fix it in ROOT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants