Closed tlambert03 closed 2 weeks ago
Attention: Patch coverage is 96.15385%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 95.60%. Comparing base (
c024a68
) to head (886a57a
).
Files | Patch % | Lines |
---|---|---|
src/useq/_position.py | 92.30% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
note @fdrgsp, this PR removes row/col from GridPosition
, not for any particular reason, but that i realized it's not actually being used here anywhere... do you have thoughts on where if anywhere those were used?
found it :)
________________________________________________________________________________
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pymmcore_widgets/_mda/_grid_widget.py", line 554, in _move_to_row_col
if pos.row == row and pos.col == col:
^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/pydantic/main.py", line 827, in __getattr__
raise AttributeError(f'{type(self).__name__!r} object has no attribute {item!r}')
AttributeError: 'RelativePosition' object has no attribute 'row'
i added back row/col ... they're not actually used anywhere, so I'm tempted to remove them, but we can leave that for another discussion. I like how this unifies the concept of positions used across useq, so would like to get this in before release. have a look when you can @fdrgsp
when looking at #166, I realized that
GridPosition
is pretty much only there to represent the concept of some "offset" relative to an absolute position. This PR renames GridPosition toRelativePosition
, and uses it more braodly