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

Materialize terror symbols #191

Conversation

smuzaffar
Copy link

No description provided.

lmoneta and others added 9 commits December 19, 2023 09:32
…d add test (root-project#6908)

* Fix in complex_quartic  problem reported in issue root-project#6900 and add test

Uncomment some different code that is used to compute the discriminat of the
resolvent cubic equation used to find roots of quartic. This code seems to eprforms better.

This fixes root-project#6900

Add tests for quartic equations

* Update and fix links and spelling in the reference documentation of ROOT::Math::Polynomial

* Increate test tolerance for fixing failure observed in  i386
The old test statistic headers and classes should not be used anymore,
since the test statistics should be created with the higher-level
functions `createNLL()` and `createChi2()`. This allows us more
flexibility in the implementation, and to phase out the old test
statistics classes eventually.
This gives us more freedom when refactoring things to also support code
generation with AD.

Also apply other code modernizations, like less use of `TString` or
delegating constructors.
As done by interpreter/llvm-project/cmake/Modules/CMakePolicy.cmake,
but this resolves many warnings when building with builtin_llvm=OFF
(but builtin_clang=ON):
```
CMake Warning (dev) at /usr/lib64/cmake/llvm/TableGen.cmake:95 (add_custom_command):
  Policy CMP0116 is not set: Ninja generators transform DEPFILEs from
  add_custom_command().  Run "cmake --help-policy CMP0116" for policy
  details.  Use the cmake_policy command to set the policy and suppress this
  warning.
Call Stack (most recent call first):
  interpreter/llvm-project/clang/cmake/modules/AddClang.cmake:25 (tablegen)
  interpreter/llvm-project/clang/include/clang/AST/CMakeLists.txt:73 (clang_tablegen)
This warning is for project developers.  Use -Wno-dev to suppress it.
```
On Windows, attempts to find R in the default installation paths if not found via find_program.
Does the same for the R_LIBRARY variable, which may not be reported by the R executable on Windows.
Adds an R_SCRIPT variable to provide the path to RScript. This is useful if the parent CMake script wants to call R non-interactively to run a script.
Fixed a typo in the comments.
nvc++ issued a warning on the last return statement being unreachable.
The variables present in TError.h are used throughout the ROOT libraries.
Anytime one of these variables is requested, e.g. with a simple test such as
`python -c "import ROOT; ROOT.kError"`, the interpreter will lookup the
corresponding symbol. Previously, the variables were declared and defined in the
header directly but were not generating any symbol since the linkage was
internal. With this commit, provide external linkage to these variables so that
the corresponding symbols are materialized in libCore.so. As a consequence, a
huge number of extra lookups by cling is avoided.
@smuzaffar
Copy link
Author

smuzaffar commented Dec 19, 2023

test parameters:

@cmsbuild
Copy link

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch cms/master/8ea6ca102c.

@smuzaffar, @iarspider, @cmsbuild, @aandvalenzuela can you please review it and eventually sign? Thanks.
@rappoccio, @sextonkennedy, @antoniovilela you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link

cmsbuild commented Dec 19, 2023

cms-bot internal usage

@smuzaffar
Copy link
Author

please test for CMSSW_14_0_ROOT6_X

@cmsbuild
Copy link

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-581391/36587/summary.html
COMMIT: f41a042
CMSSW: CMSSW_14_0_ROOT6_X_2023-12-18-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/root/191/36587/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-581391/36587/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-581391/36587/git-merge-result

Comparison Summary

Summary:

@smuzaffar
Copy link
Author

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

Successfully merging this pull request may close these issues.

9 participants