Skip to content

[Feat]: Same timestep used for all validation steps #772

Description

@dxqb

What happened?

In this PR, the validation timestep was fixed to 499: #660
This was to make sure each validation uses the same timesteps, and therefore make the validations comparable to each other. Validation loss became meaningful.

I don't think this is the right solution though. The timesteps should be the same for each validation, but not for each validation step within the same validation.

Only validating against 499 might be somewhat meaningful, but especially in models with timestep shifting, 499 is not a particularily important timestep.

What did you expect would happen?

Validation timesteps should be deterministic, but only between validations, not between validation steps. Within the same validation, timesteps should be randomly distributed.

Relevant log output

Generate and upload debug_report.log

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions