-
Notifications
You must be signed in to change notification settings - Fork 52
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
MASSIF-1 Crystal Direct Harvester implementation #826
Conversation
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.
A couple of random comments. I did not review the whole thing, we do not seem to have such harvester.
d25f603
to
ef16466
Compare
28216eb
to
44754ad
Compare
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.
Surely a file called EMBLFlexHCD.py should be in HardwareObjects/EMBL, and not in HardwareObjects/ ?
Not quite @rhfogh, in this case it refers that the flex is manifactured by EMBL, just as Cats, Marvin ...:-) |
40aa854
to
c58b0f3
Compare
c58b0f3
to
20689e7
Compare
8372495
to
2000bc8
Compare
2000bc8
to
4469cda
Compare
@rhfogh do you still request this change ? |
1fac006
to
324c0a8
Compare
Looks good as far as I can see :), (its a big one ;) ) |
Hi @jbflo, Most of the changes are Ok with me. It still bothers me, though, that you have to check for harvester in the EMBLFlexHCD.py. |
Add default timeout 0
Split method and fix flake8 warnning
crystal_symmetry and yaml where imported twice
Remove hard coded key
Add Test data to Mock up / Fix Type Hint
which inherits from the EMBLFlexHCD
Remove Duplicated line
Should be AttributeError, IndexError
Removed from UI
Apply Spell Check has_lims_data is used on qt version can't remove but should be mark as Deprecated
54143a6
to
4278ad2
Compare
This code related to the implementation of the Crystal Direct Harvester in Massif-1 ESRF
This should not have breaking change.
Beamline who does not have an Harvester should not have any new things/modification to do
A PR in Mxcube-web related to this one need to be reviewed as well .