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.

Sign up for GitHub

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

Jump to bottom

Handling acronyms in ALF spec #80

Open
2 of 3 tasks
k1o0 opened this issue Feb 17, 2023 · 1 comment
Open
2 of 3 tasks

Handling acronyms in ALF spec #80

k1o0 opened this issue Feb 17, 2023 · 1 comment

Comments

@k1o0
Copy link
Collaborator

k1o0 commented Feb 17, 2023 •
edited
Loading

How should we name ALF objects with acronyms and initialisms?
Current behaviour:

one.alf.spec.to_alf('DAQ data', 'wiring', 'json')
>> 'DAQData.wiring.json'

Currently we have ROIMotionEnergy.position, DAQdata.raw.

Suggestion:

  • 'DAQdata' should be renamed to 'DAQData' in order to remain standard.
  • Add function to display and ALF part with whitespaces.
  • handle plural acronyms, e.g. mpci ROIs
@k1o0 k1o0 changed the title modify spec for camel case acronymes in ALF Handling acronyms in ALF spec Feb 17, 2023
@k1o0
Copy link
Collaborator Author

k1o0 commented Feb 17, 2023

a6805f2

k1o0 added a commit that referenced this issue Nov 13, 2024
@k1o0
str.lower -> str.casefold; support for plural acronyms (issue #80)
ecf3caa
k1o0 added a commit that referenced this issue Nov 13, 2024
@k1o0
str.lower -> str.casefold; support for plural acronyms (issue #80)
b3b4d79
k1o0 added a commit that referenced this issue Nov 13, 2024
@k1o0
str.lower -> str.casefold; support for plural acronyms (issue #80)
55ce32a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@k1o0