-
Notifications
You must be signed in to change notification settings - Fork 4
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
add first-class support for discrete hardware #98
Conversation
yaqc_bluesky/_is_discrete.py
Outdated
out = super()._describe(out) | ||
meta = OrderedDict() | ||
meta["shape"] = [] | ||
meta["dtype"] = "string" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The identifier can be null, not sure if that can be declared here or needs to be handled, e.g. by mapping None to empty string
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great point. I guess I'll have to try and feed nulls into Bluesky to see what happens. If Bluesky gets angry we can do empty string easily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
None
works just fine, with the exception of formatting for best effort callback. The messages get emitted and data gets recorded.
I guess now we have to decide, but I think keeping it as-is makes sense.
* setter not getter * tests * rm print
No description provided.