make_exception_fields
Could you update the commit message? String is already a primitive type, and AFAIK you're just adding support for the
str
attribute. I will change the commit message to "nac3artiq: support…
Splitting PR for Clarity I have split the PR into two separate PRs for better clarity and management.
- PR #416: This PR focuses on supporting string attributes
- PR #417: This PR…
From what I have tested, only the last element inside the class definitions was locked. I was unable to see what it corresponded to in the class, but methods including the constructor were already…
Added this at the top for clarity in if condition, but yes since it is used only once, will rewrite this and move it with the calling condition.
The previous code entered deadlock since the last element inside the def was locked (not sure about the reason for this), so to avoid that, I replaced the read() function with try_read to…