cbalint13 commented on PR #14468:
URL: https://github.com/apache/tvm/pull/14468#issuecomment-1497916664
@tqchen ,
> @cbalint13 Yes, let us update and to change all to reg.
* Updated all tutorials, apps and tests from any explicit "rank" to "reg".
> cc @junrushao to double check cases in MetaSchedule.
* Double check is welcome, as quick info, none of my metascheduler test
failed, xgboost would print:
```
xgboost.core.XGBoostError: [15:59:59]
/builddir/build/BUILD/xgboost/src/common/ranking_utils.h:378:
Check failed: is_binary: MAP can only be used with binary labels.
```
* Used xgboost 20230403 git hash 15e073ca .
> Might be useful to use reg:logistic, if the output is scaled into [0, 1]
My thought on the newly introduced ```type_loss="rank-binary"```:
* Is **alternative** to the original ```rank``` and **booth have**:
```("objective": "rank:pairwise")```
* It binarize (within a cloned copy) the labels only for the xgboost eval
step (being xgboost>=2 compatible).
* The dynamics inside TVM further steps still remained with full
continuous values.
I would leave it as described, if would like we can create another one
```type_loss="reg-binary" w/ reg:logistic```.
For the case reg:logistic the naming "rank-binary" loose the "rank"-ness, so
it would be just another "reg" class type.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
