-
Notifications
You must be signed in to change notification settings - Fork 1
Mocksim merge #18
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
base: main
Are you sure you want to change the base?
Mocksim merge #18
Changes from 11 commits
7625d0b
addba05
0cca383
ecbcd95
1bc6fa1
c173639
4900969
3be68e2
1761738
c661925
8e35419
1f7caed
f14092a
3eae984
903a987
3f41e1e
9649654
01457ef
3f5e0d5
cb0268b
d089a40
e84a6c5
a44d570
c29c569
7a86da3
c5fe533
6e8c8dc
5115afa
8b23e35
fd44984
30f1557
e780f01
b67fb6b
948cf91
762dba2
0e0461e
77cb9a4
70bc5c0
a7e2623
56ca49a
df1f28f
5cfaf2c
adfe9f6
f469b67
11b9994
ff8f580
ca3fd15
c98ff9d
83843d0
667ae12
1f29ae7
701bd16
017eebb
cf77561
7009cfa
57f56ac
ea383e3
4896a39
6cedd8e
be1ebfa
ef6e030
b4a7397
04efbb1
6ee3d7c
73e4e1a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,8 +10,8 @@ | |
"show_axes": True, | ||
"auto_select_corr_points": True, | ||
}, | ||
"calibration_path": "./calibrations", | ||
"console_history_file": "./console_history", | ||
"calibration_path": "./", | ||
"console_history_file": "./", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. console_history is a file; this should be left as-is |
||
"console_edit_command": "code -g {fileName}:{lineNum}", | ||
} | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -69,11 +69,11 @@ def __init__(self, model): | |
self.dropdown.activated.connect(self.handle_stage_selection) | ||
|
||
self.settings_button = QPushButton() | ||
self.settings_button.setIcon(QIcon('../img/gear.png')) | ||
self.settings_button.setIcon(QIcon('./img/gear.png')) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. More robust:
|
||
self.settings_button.clicked.connect(self.handle_settings) | ||
|
||
self.calibration_label = QLabel("") | ||
self.cal_pt_btn = QPushButton('') | ||
self.cal_pt_btn = QPushButton('Copy Calibration Point') | ||
self.cal_pt_btn.clicked.connect(self.copy_cal_pt) | ||
|
||
self.xcontrol = AxisControl('x') | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -79,7 +79,6 @@ def list_calibrations(self): | |
for cal in self.calibrations.values(): | ||
calibrations.append({'calibration': cal, 'from_cs': cal.from_cs, 'to_cs': cal.to_cs, 'timestamp': cal.timestamp}) | ||
|
||
assert len(calibrations) > 0 | ||
return calibrations | ||
|
||
def get_calibration(self, stage): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this gets the most recent calibration associated with a stage There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. used for "move to selected" button |
||
|
@@ -141,18 +140,29 @@ def add_transform(self, name, transform): | |
def get_transform(self, name): | ||
return self.transforms[name] | ||
|
||
def set_lcorr(self, xc, yc): | ||
self.lcorr = [xc, yc] | ||
|
||
def clear_lcorr(self): | ||
self.lcorr = False | ||
|
||
def set_rcorr(self, xc, yc): | ||
self.rcorr = [xc, yc] | ||
|
||
def clear_rcorr(self): | ||
self.rcorr = False | ||
|
||
def handle_accutest_point_reached(self, i, npoints): | ||
self.msg_posted.emit('Accuracy test point %d (of %d) reached.' % (i+1,npoints)) | ||
self.clear_lcorr() | ||
self.clear_rcorr() | ||
self.msg_posted.emit('Highlight correspondence points and press C to continue') | ||
|
||
def register_corr_points_accutest(self): | ||
lcorr, rcorr = self.lcorr, self.rcorr | ||
if (lcorr and rcorr): | ||
self.accutest_worker.register_corr_points(lcorr, rcorr) | ||
self.msg_posted.emit('Correspondence points registered: (%d,%d) and (%d,%d)' % \ | ||
(lcorr[0],lcorr[1], rcorr[0],rcorr[1])) | ||
corr_pt = self.get_image_point() | ||
if corr_pt is not None: | ||
self.accutest_worker.register_corr_points(corr_pt) | ||
self.msg_posted.emit('Correspondence points registered.') | ||
self.accutest_worker.carry_on() | ||
else: | ||
self.msg_posted.emit('Highlight correspondence points and press C to continue') | ||
|
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.
Do you prefer calibrations here, or was tere another reason to make this change?