scipp / scipp/ess

Time-of-flight: do we still need to rename to tof?

Open
#565 0 comments 0 reactions 1 assignee View on GitHub

@jl-wynen is already working on this.

Since Feb 9, 2026.

essspectroscopy
Dominant language
Python
Stars
1
Forks
3
Avg merge
2d 11h
Merged PRs (30d)
17

Description

ess/spectroscopy/indirect/time_of_flight.py:84: in monitor_time_of_flight_data
monitor_data=monitor_data.rename(t='tof'),

Is this still needed? We need to check that the downstream code can work without the rename?
The dimension could either be t or event_time_zero in newer files.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.