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

[RF] Fix tutorial category of rf206_treevistools #18029

Merged
merged 1 commit into from
Mar 18, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion tutorials/roofit/roofit/rf206_treevistools.C
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/// \file
/// \ingroup tutorial_roofit
/// \ingroup tutorial_roofit_main
/// \notebook -nodraw
/// Addition and convolution: tools for visualization of RooAbsArg expression trees
///
2 changes: 1 addition & 1 deletion tutorials/roofit/roofit/rf206_treevistools.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
## \file
## \ingroup tutorial_roofit
## \ingroup tutorial_roofit_main
## \notebook -nodraw
## Addition and convolution: tools for visualization of ROOT.RooAbsArg expression trees
##

Unchanged files with check annotations Beta

if (isOffset()) {
assert(Source &&
"Cannot deserialize a lazy pointer without an AST source");
Ptr = reinterpret_cast<uint64_t>((Source->*Get)(Ptr >> 1));

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / debian125 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20, dev=ON

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora40 LLVM_ENABLE_ASSERTIONS=On, CMAKE_CXX_STANDARD=20

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]

Check warning on line 378 in interpreter/llvm-project/clang/include/clang/AST/ExternalASTSource.h

GitHub Actions / fedora41 LLVM_ENABLE_ASSERTIONS=On

‘this’ pointer is null [-Wnonnull]
}
return reinterpret_cast<T*>(Ptr);
}
zmemzero((uint8_t *)s->head, (unsigned)(s->hash_size)*sizeof(*s->head));
/* ========================================================================= */
int ZEXPORT deflateInit_(strm, level, version, stream_size)

Check warning on line 1243 in builtins/zlib/deflate_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
z_streamp strm;
int level;
const char *version;
#define DO8 DO1; DO1; DO1; DO1; DO1; DO1; DO1; DO1
/* ========================================================================= */
local unsigned long crc32_generic(crc, buf, len)

Check warning on line 243 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
unsigned long crc;
const unsigned char FAR *buf;
uInt len;
return crc ^ 0xffffffffUL;
}
uLong crc32(crc, buf, len)

Check warning on line 277 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
uLong crc;
const Bytef *buf;
uInt len;
#define DOLIT32 DOLIT4; DOLIT4; DOLIT4; DOLIT4; DOLIT4; DOLIT4; DOLIT4; DOLIT4
/* ========================================================================= */
local unsigned long crc32_little(crc, buf, len)

Check warning on line 294 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
unsigned long crc;
const unsigned char FAR *buf;
unsigned len;
#define DOBIG32 DOBIG4; DOBIG4; DOBIG4; DOBIG4; DOBIG4; DOBIG4; DOBIG4; DOBIG4
/* ========================================================================= */
local unsigned long crc32_big(crc, buf, len)

Check warning on line 334 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
unsigned long crc;
const unsigned char FAR *buf;
unsigned len;
#define GF2_DIM 32 /* dimension of GF(2) vectors (length of CRC) */
/* ========================================================================= */
local unsigned long gf2_matrix_times(mat, vec)

Check warning on line 374 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
unsigned long *mat;
unsigned long vec;
{
}
/* ========================================================================= */
local void gf2_matrix_square(square, mat)

Check warning on line 391 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
unsigned long *square;
unsigned long *mat;
{
}
/* ========================================================================= */
local uLong crc32_combine_(crc1, crc2, len2)

Check warning on line 402 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
uLong crc1;
uLong crc2;
z_off64_t len2;
}
/* ========================================================================= */
uLong ZEXPORT crc32_combine(crc1, crc2, len2)

Check warning on line 458 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
uLong crc1;
uLong crc2;
z_off_t len2;
return crc32_combine_(crc1, crc2, len2);
}
uLong ZEXPORT crc32_combine64(crc1, crc2, len2)

Check warning on line 466 in builtins/zlib/crc32_cf.c

GitHub Actions / fedora42 LLVM_ENABLE_ASSERTIONS=On

old-style function definition [-Wold-style-definition]
uLong crc1;
uLong crc2;
z_off64_t len2;
// Generate code for Session data members (e.g. internal vectors)
virtual std::string GenerateSessionMembersCode(std::string opName) override {
size_t outputChannelSize = fShapeY[2]; // size/channel = D * H * W

Check warning on line 306 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'outputChannelSize' set but not used [-Wunused-but-set-variable]

Check warning on line 306 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'outputChannelSize' set but not used [-Wunused-but-set-variable]

Check warning on line 306 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'outputChannelSize' set but not used [-Wunused-but-set-variable]
size_t kernelSize = fAttrKernelShape[0];

Check warning on line 307 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'kernelSize' set but not used [-Wunused-but-set-variable]

Check warning on line 307 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'kernelSize' set but not used [-Wunused-but-set-variable]

Check warning on line 307 in tmva/sofie/inc/TMVA/ROperator_Conv.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

variable 'kernelSize' set but not used [-Wunused-but-set-variable]
for (size_t i = 1; i < fDim; i++) {
outputChannelSize *= fShapeY[2 + i];
kernelSize *= fAttrKernelShape[i];
model.AddNeededStdLib("algorithm");
}
std::string Generate(std::string opName){

Check warning on line 123 in tmva/sofie/inc/TMVA/ROperator_Reduce.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

'Generate' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]

Check warning on line 123 in tmva/sofie/inc/TMVA/ROperator_Reduce.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

'Generate' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]

Check warning on line 123 in tmva/sofie/inc/TMVA/ROperator_Reduce.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

'Generate' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]

Check warning on line 123 in tmva/sofie/inc/TMVA/ROperator_Reduce.hxx

GitHub Actions / alma9-clang clang LLVM_ENABLE_ASSERTIONS=On, CMAKE_C_COMPILER=clang, CMAKE_CXX_COMPILER=clang++

'Generate' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
opName = "op_" + opName;
if (fShapeX.empty() || fShapeY.empty()) {
throw std::runtime_error("TMVA SOFIE Reduce Op called to Generate without being initialized first");