Sum dataset

[LuisFSegalla]

• Depends on Pause dev #1

Added the sum of all N channels connected to the detector as a dataset.

[GDYendell]

I am not sure about the reformatting. I think it best to just stick with the current formatting unless we implement some tooling for format checking in CI and ideally auto formatting, because otherwise it will have to manually reviewed every time.

[LuisFSegalla]

Do you think we could add something like a settings file on .vscode that's standard for all detectors and use it from now on? I'm guilty on always setting the autoformat to false, but will stop from now on.

[GDYendell]

I think just putting it in .vscode is not going to be sufficient. If it isn't strictly checked by a CI step then someone has to review the formatting and it massively increases the work required to review and distracts from the actual code changes. If we can do that and standardise it across all the repos (including odin-data) then that would be great, but until then I think just try to keep it consistent with the current formatting.

Will it be difficult to pull out the formatting changes from this PR?

[LuisFSegalla]

Hi Gary,
It should be fine to remove the formatting, I deleted that commit from a local branch and it seemed to be okay without it. will do some extra checks to make sure it didn't affect anything else and will push the changes.

[GDYendell]

OK that looks better without the formatting of the existing code, but could you also match the formatting of the existing code with the new code, e.g. indentation does not match.