Skip to content

saving: Dash is a valid char

Valentin Valls requested to merge fix-beamline-name into master

id06-lvp is a valid name but was hidenly renamed into id06lvp

         RuntimeError: Cannot create directory (Nexus writer OSError: [Errno 30] Read-only file system: '/data/id06lvp')

This PR only fixes this specific char, but notice that the underscore _ is also removed.

I also don't understand the logic here.

The beamline name have to be valid or have to be rejected and fixed manually. There is no reason to patch it underneath. This makes the problems really difficult to diagnosis.

The previous line contains a match.

Why don't we just trust the match?

Edited by Valentin Valls

Merge request reports