chore: allow skip pure torch computation on benchmark camelyon

[ThibaultFy]

Related issue

part of FL-1507

Summary

Notes

Please check if the PR fulfills these requirements

• If the feature has an impact on the user experience, the changelog has been updated
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• The commit message follows the conventional commit specification

[linear]

FL-1507 Make pure torch workflow optional

[thbcmlowk]

Thank you!

[thbcmlowk]

To avoid negation in the if statement and keep the default behavior at the first indentation level, I think I'd rather do:

Suggested change

	if not exp_params["skip_pure_torch"]:
	torch_metrics = torch_fed_avg(
	train_folder=train_folder,
	test_folder=test_folder,
	**{k: v for k, v in exp_params.items() if k in run_keys},
	index_generator=index_generator,
	model=model,
	)
	results = {**exp_params, **{"results": {**substrafl_metrics.to_dict, **torch_metrics.to_dict}}}
	load_benchmark_summary(file=LOCAL_RESULTS_FILE, experiment_summary=results)
	assert_expected_results(substrafl_metrics=substrafl_metrics, torch_metrics=torch_metrics, exp_params=exp_params)
	if exp_params["skip_pure_torch"]:
	return
	torch_metrics = torch_fed_avg(
	train_folder=train_folder,
	test_folder=test_folder,
	**{k: v for k, v in exp_params.items() if k in run_keys},
	index_generator=index_generator,
	model=model,
	)
	results = {**exp_params, **{"results": {**substrafl_metrics.to_dict, **torch_metrics.to_dict}}}
	load_benchmark_summary(file=LOCAL_RESULTS_FILE, experiment_summary=results)
	assert_expected_results(substrafl_metrics=substrafl_metrics, torch_metrics=torch_metrics, exp_params=exp_params)

But it is basically the same so either way is fine by me

[thbcmlowk]

Thanks for updating!