Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] warning for long assay / study titles #126

Closed
Brilator opened this issue Apr 13, 2022 · 2 comments · Fixed by #170
Closed

[Feature Request] warning for long assay / study titles #126

Brilator opened this issue Apr 13, 2022 · 2 comments · Fixed by #170

Comments

@Brilator
Copy link
Member

Is your feature request related to a problem? Please describe.
Excel bugs with long (>31 characters) sheet names.

Describe the solution you'd like
Just a quick warning to reduce titles to max. 31 chars and preferentially without specials and spaces.

Additional context

Try

arc s add -s 'This Is a very long Title that Excel does not like'

see Excel error (at least on Mac):

"We found a problem with some content in 'isa.study.xlsx'. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes."

@HLWeil
Copy link
Member

HLWeil commented Apr 14, 2022

Hey @Brilator, good point! Besides this length constraint, also some other constraints like characters that clash with the different OS path names need to be blocked for some arguments. E.g. for windows:
image

@muehlhaus, @chgarth, @omaus, maybe we could collect similar constraints here in this issue?

@omaus
Copy link
Collaborator

omaus commented Apr 14, 2022

Hey @Brilator, good point! Besides this length constraint, also some other constraints like characters that clash with the different OS path names need to be blocked for some arguments. E.g. for windows: image

@muehlhaus, @chgarth, @omaus, maybe we could collect similar constraints here in this issue?

(Space) is difficult on Linux and should be avoided (though it is not forbidden). Maybe replacing with _ when OS is Linux?
Only / and NUL are forbidden in Linux. Don't know for Mac. @Brilator ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants