MVAUtils: Adding support for XGBoost

Adding derived class to support XGboost node/forest in MVAUtils. A python utility to convert the model saved by XGBoost training to our TTree format is also provided. Implementation follows closely with what was done in lgbm, presently only support regression and binary classification.

added 21.2 label

This merge request affects 1 package:

• Reconstruction/MVAUtils

Adding @goetz ,@turra as watchers

added Reconstruction review-pending-level-1 labels

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash abd096be)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 10
For experts only: Jenkins output [CI-MERGE-REQUEST 42297]

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash abd096be)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 3
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8329]

added Egamma label

Mentioning @turra

added 1 deleted label

added 1 commit

• 6d170bfe - Minor update in README

Compare with previous version

This merge request affects 1 package:

• Reconstruction/MVAUtils

Adding @goetz ,@turra as watchers

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 6d170bfe)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 42302]

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 6d170bfe)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 3
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8345]

Please resolve discussion threads and relabel. Magdalena (L1)

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• c5cabe3f - Update copyright statement to 2020

Compare with previous version

This merge request affects 1 package:

• Reconstruction/MVAUtils

Adding @goetz ,@turra as watchers

added review-pending-level-1 label and removed review-user-action-required label

resolved all threads

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash c5cabe3f)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 42307]

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash c5cabe3f)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 3
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8381]

Can you also update the tests? This: https://gitlab.cern.ch/atlas/athena/blob/master/Reconstruction/MVAUtils/test/ut_test_MVAUtils.py

added review-user-action-required label and removed review-pending-level-1 label

Removing the L1 review label until internal discussions are solved. Feel free to put it back once all discussions are resolved (will be put back by the CI in case there are new commits).

I also saw some warnings in the CI but I don't think they are related to this MR.

Kira (L1)

added 2 commits

• 99917f2b - Change to better understanding splitting condition
• 3334a0a2 - Add XGBoost unit test modules

Compare with previous version

This merge request affects 1 package:

• Reconstruction/MVAUtils

Adding @goetz ,@turra as watchers

added review-pending-level-1 label and removed review-user-action-required label

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 3334a0a2)

	AnalysisBase	AthAnalysis
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 8522]

Builds against 21.2 are performed on slc6 (for AthDerivation) and on CentOS7 (for other projects). Two result notes are posted.

• Developers and shifters need to wait for both slc6(AthDerivation) and CentOS7(other 3 projects) results, sometimes for additional hour or two.

CI Result SUCCESS (hash 3334a0a2)

	AthDerivation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
AthDerivation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 42348]

resolved all threads

Hi, I would like to pass it to L2 shifter in order to double check the new full files that have been added.

Thanks, Mazuza (L1)

added review-user-action-required label and removed review-pending-level-1 label

added review-pending-level-2 label and removed review-user-action-required label