Skip to content
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

Why is subtraction used in _XACTUAL calculation #1

Closed
orangeccc opened this issue Aug 1, 2022 · 2 comments
Closed

Why is subtraction used in _XACTUAL calculation #1

orangeccc opened this issue Aug 1, 2022 · 2 comments

Comments

@orangeccc
Copy link

hi,
Good work!
I got a question about your code.
Why is subtraction used in _XACTUAL at

df['_XACTUAL'] = (df['XINDEX'] * self.die_pitch[0]) + df['XREL'] - self.center_location[0]
df['_YACTUAL'] = (df['YINDEX'] * self.die_pitch[1]) + df['YREL'] - self.center_location[1]

I think maybe it should be
df['_XACTUAL'] = (df['XINDEX'] * self.die_pitch[0]) + df['XREL'] + self.center_location[0]
df['_YACTUAL'] = (df['YINDEX'] * self.die_pitch[1]) + df['YREL'] + self.center_location[1]?

@orangeccc
Copy link
Author

oh, I think i got it.
is the center location means the wafer center coordinate with 0, 0 die as origin?

and i found that the die lines in the output result map is different with another [software's.].
(https://slriley.com/software/klarfview-overview-2/)
1a

1b
(Note the points and lines on the left.)

after i edit the line

die_x_lines = self.center_location[0] + (i * self.die_pitch[0])

and
die_y_lines = self.center_location[1] + (i * self.die_pitch[1])

to
die_x_lines = -self.center_location[0] + (i * self.die_pitch[0])
die_y_lines = -self.center_location[1] + (i * self.die_pitch[1]),
then they are exactly the same.

2
1b

so maybe there shoud be a minus in die lines calculation?

MichaelHotaling added a commit that referenced this issue Aug 8, 2022
Issue Number: #1
Found issue with scribe lines not aligning with third party software
Resolved by adding negative to both `die_x_lines` and `die_y_lines`
Thank you, orangeccc for the opening this bug report.
@MichaelHotaling
Copy link
Owner

Nice find!
I've added the changes into the code base.

df87274

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

No branches or pull requests

2 participants