evaluate_sync method on Dataset should support retry_task and retry_evaluators arguments

[billiejoe-bw]

Description

evaluate_sync on the Dataset class (for evals) is a

synchronous wrapper around evaluate provided for convenience

but currently it cannot forward the retry_task and retry_evaluators arguments that are needed to supply retry configuration. It would be more convenient if it did that.

(I'm happy to make a PR for that if it's helpful btw; looks like a small change.)

References

• evaluate_sync method
• evaluate method

[DouweM]

@billiejoe-bw You're right, the sync methods should accept all the same arguments as the async ones they're based on -- a PR would be much appreciated.