Skip to content

Implementation of OB FPC Capacitors - #15652

Open
mario6829 wants to merge 2 commits into
AliceO2Group:devfrom
mario6829:itsgeo
Open

Implementation of OB FPC Capacitors#15652
mario6829 wants to merge 2 commits into
AliceO2Group:devfrom
mario6829:itsgeo

Conversation

@mario6829

Copy link
Copy Markdown
Contributor

The capacitors on the OB FPC have been implemented. Dimensions and positions were obtained from Gerber files and direct information from experts. It has been verified that the sensor position hasn't changed after the implementation.

@f3sch f3sch 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.

LGTM!

I guess we are effectively not using the mBuildLevel otherwise for buildLevel >= 5 the ypos would not be modified and the capacitors are placed ~150 um below their nominal position?

P.S. Did you check the potentially floating capacitors, I pointed out a while ago (maybe I was also wrong)?

@mario6829

Copy link
Copy Markdown
Contributor Author

Hi @f3sch indeed in normal run mBuildLevel is always equal to 0. It is (well, actually was) used only for material budget studies, in order to determine the contribution of each material by selectively taking them out. So under any normal execution it has no effect. Nevertheless I acknowledge that the logic was corrected before, but now it can be improved. I'll fix it in the next release.
The check on the potentially floating capacitors is the very next task in my to-do list. I'll come back shortly with a result (either a negative check or a code fix). Thanks for pointing it out.
Best

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants