Skip to content

Clarification on TVD loss definition in distillation #3

Description

@shqii1j

Hi, thanks for releasing the code and paper.

I have a question regarding the TVD loss used in the distillation objective. From the code, both preds and target appear to be log-probabilities. However, in sdtt/src/sdtt/core/distill/multi_round_sdtt.py (line 349), the TVD loss is implemented as:

(preds - target).abs().sum(-1).mean()

This seems to correspond to an L1 distance in log-probability space, rather than the standard total variation distance defined in the Appendix B.

Could you confirm whether this is an intentional surrogate loss (log-probability L1), and if so, what the motivation is compared to computing TVD directly in probability space?

Thank you!

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions