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.

Sign up for GitHub

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

Jump to bottom

FIX - Records attributes missing #11

Open
mmdonatti opened this issue May 16, 2023 · 8 comments
Open

FIX - Records attributes missing #11

mmdonatti opened this issue May 16, 2023 · 8 comments

Comments

@mmdonatti
Copy link
Collaborator

DevEnbl-Sel must have PINI attribute
@mmdonatti mmdonatti changed the title FIX - DevEnbl-Sel record init FIX - Records missing initialization attribute May 25, 2023
@mmdonatti
Copy link
Collaborator Author

EVEs OTP Enable-Sel is also missing. Other devices must be checked

@mmdonatti
Copy link
Collaborator Author

EVG interlock map enable is also missing.

@mmdonatti
Copy link
Collaborator Author

Add Hz unit to FPGAClk-Cte EVE/EVR records

@mmdonatti mmdonatti changed the title FIX - Records missing initialization attribute FIX - Records attributes missing Jun 6, 2023
@mmdonatti
Copy link
Collaborator Author

Check EVR/EVEs DigitalInputs Enable

@mmdonatti
Copy link
Collaborator Author

Add ClkDiv to EVG autosave

@ericonr
Copy link
Member

ericonr commented Aug 22, 2023

EVEs OTP Enable-Sel is also missing. Other devices must be checked

Is that the $(P)$(R)OTP$(num)State-Sel record?

EVG interlock map enable is also missing.

Which record is this? $(P)$(R)IntlkTbl0to15-Sel and $(P)$(R)IntlkTbl16to27-Sel?

Check EVR/EVEs DigitalInputs Enable

It is missing from them.

Fixing some of these cases is making me want to refactor the DB files into the files with the common definitions for each type of property, because editing everything is such a pain... I think we should maybe rethink fixing all of these for september.

@ericonr ericonr mentioned this issue Aug 22, 2023
Merged
@mmdonatti
Copy link
Collaborator Author

Is that the $(P)$(R)OTP$(num)State-Sel record?

Yes! 🙄

Which record is this? $(P)$(R)IntlkTbl0to15-Sel and $(P)$(R)IntlkTbl16to27-Sel?

$(P)$(R)IntlkCtrlEnbl-Sel

@ericonr
Copy link
Member

ericonr commented Aug 24, 2023

Add ClkDiv to EVG autosave

Do you mean MuxDiv-SP? It already is, and has PINI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mmdonatti @ericonr