Skip to content

Update scanTarget_StageRepeatability - #6

Open
alejoruge wants to merge 2 commits into
masterfrom
diego
Open

alejoruge wants to merge 2 commits into
masterfrom
diego

Conversation

@alejoruge

Copy link
Copy Markdown

Follow the same logic and perform calibration properly.

Update to follow the same logic and perform calibration properly.
@alejoruge
alejoruge requested a review from MyYo July 3, 2025 23:10
@alejoruge

Copy link
Copy Markdown
Author

Quick question: I am not sure why for the second pattern (Pattern 2) from line 72 to line 115, it was not using the function yOCTPhotobleachTile. I followed the same logic but just wondering if we should. I ensured it works as is

@MyYo MyYo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks, a few changes requested

- skipHardware default set to false
- Standard notation for octProbePath from demos
@alejoruge
alejoruge requested a review from MyYo July 10, 2025 10:21

@MyYo MyYo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor change thank you

nPasses = 4; % Keep as low as possible. If galvo gets stuck, increase number

%% Define pattern template
[~, fileName, ~] = fileparts(octProbePath); % Extract filename from path to get lens magniciation

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sorry, I didn't mean for you to extract lens from file parts
You can instead use:
lens = '40x'
octProbePath = yOCTGetProbeIniPath(lens,'OCTP900'); % Inputs to the function are OBJECTIVE_DEPENDENT: '10x' or '40x', and scanning system dependent 'OCTP900' or ''

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.

2 participants