Skip to content

Conversation

@brookeferber
Copy link
Contributor

No description provided.

@CJ-Wright CJ-Wright self-requested a review July 24, 2017 16:06
.ropeproject .gitignore
xpdsim/robot.py Outdated
self._current_sample_geometry = None
super().__init__(name, fields, initial_set, **kwargs)

def load_sample(self, saple_number, sample_geometry=None):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling

xpdsim/dets.py Outdated


def build_image_cycle(path):
# Goal: replace this with a mediator to the robot, to be handled in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The image cycle stuff is fine. The issue is how that image cycle is handed to the detector

@codecov-io
Copy link

codecov-io commented Jul 26, 2017

Codecov Report

Merging #27 into master will decrease coverage by 26.26%.
The diff coverage is 1.61%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #27       +/-   ##
===========================================
- Coverage   93.54%   67.28%   -26.27%     
===========================================
  Files           6        7        +1     
  Lines         155      217       +62     
===========================================
+ Hits          145      146        +1     
- Misses         10       71       +61
Impacted Files Coverage Δ
xpdsim/robot.py 0% <0%> (ø)
xpdsim/dets.py 85.52% <25%> (-3.37%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update adfa6d7...a522f44. Read the comment docs.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants