Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
add i18 beamline definition - WITH device_factory #722
add i18 beamline definition - WITH device_factory #722
Changes from 41 commits
0baece2
c902167
4fb6a76
7311a8e
9c52087
d902185
d774413
ed4f4c8
52d6ffe
42f1618
ba01d19
80ac2e8
2aa1255
378b978
d2a5c14
a01e7f5
e97bbae
1792482
1502f89
5b760eb
a712008
b0583a8
52f7b71
ccd69e5
0f53ba6
68c0a7a
a8a020b
314edda
9d522ca
9376dd1
67b43cd
f5f0618
b808dd1
9f2369e
6526c61
01de971
70df1eb
78fdc27
5224b96
61007bd
e8c8bcd
e039885
d94415f
5972984
ebb5175
1068b44
9201e5b
b916855
216728c
1ba4df5
cdaaf25
7d2ddd3
c19a147
9faa28e
eb13245
02a94dd
fd7adc2
6616168
2dccb09
f9bf3e5
3f4cbf2
18515b6
f1e03f9
87c9c73
b79b76d
11b08e3
f743462
e753e40
17fa5d3
7c651cc
e3a2e0e
3b3fc28
5536626
568828d
8e42ad9
8e17e2f
c239359
1e7f4a2
5fe2ca2
fe7ff5e
a3f8b1a
b5fef2d
05f2f50
5b8fec4
3f2a7c3
7d66522
cfc9f78
5a87cee
f111987
5fbf51e
10b54a7
8171e96
de2d3c6
5f4220e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Must: make an issue and note it here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I'd prefer to see it taken out entirely, if "not ready anytime soon" and waiting on a controls project.
Check warning on line 129 in src/dodal/beamlines/i18.py
src/dodal/beamlines/i18.py#L129
Check warning on line 136 in src/dodal/beamlines/i18.py
src/dodal/beamlines/i18.py#L136
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these crystals correct or copied from i22?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can double check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I sent an email about this
Check warning on line 144 in src/dodal/beamlines/i18.py
src/dodal/beamlines/i18.py#L144
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should: this can be unskipped.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Must: note why this is skipped: #1009
Check warning on line 194 in src/dodal/beamlines/i18.py
src/dodal/beamlines/i18.py#L194
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: alternative form
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: not used anywhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to have it for reference for checking the object status though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit
Check warning on line 24 in src/dodal/devices/i18/thor_labs_stage.py
src/dodal/devices/i18/thor_labs_stage.py#L23-L24