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
473 create a device for xpress3 areadetector #524
473 create a device for xpress3 areadetector #524
Changes from 33 commits
b87b45d
7000811
ff04eac
29dc4da
9c80e3b
4b021db
0ee8d5e
da6174b
630f8f2
c2a37a9
cb3d02d
e120df7
4c46d52
a7208a3
22d721e
b31a03f
404c52b
67fcb8d
f6a829e
2bb0ac5
1adba5d
fc0f508
d757e40
f24095f
3997024
66a4a28
4921c36
492084b
9fcd060
96b2c73
76b3646
df27b1a
7b0204a
9527f20
152132d
17a9228
8f3b0d2
2897a05
5a1acf8
3f5f53f
6500e57
4d6a3e8
a93e60d
489e8ca
5c1e703
a3787c3
f5cb74e
12335d0
97c91be
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.
Should: I don't think it makes sense to wait on both of these
stage
basically means get the device ready to take data so I expect that we want to wait forACQUIRE
(it is now taking data) rather thanDONE
(it has finished taking data).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 think I got confused with the return await_value(self.acquire, 0).
would this be more correct with unstage?
Or just does nothing?
I think RE automatically call unstage if stage was used, so if we use stage we will need to have unstage. The most annoying part is RE seem to just skip and make it looks like test is passing if there is no unstage.....
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 think the waiting for the done in
unstage
makes more logically sense to me as something we would expect on the device. So let's leave it at that and make sure it works as expected as part of https://github.com/DiamondLightSource/hyperion/issues/1433This file was deleted.
This file was deleted.