-
Notifications
You must be signed in to change notification settings - Fork 78
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
ENH: ArchivePlotCurveItem Connection Status #1108
ENH: ArchivePlotCurveItem Connection Status #1108
Conversation
4db392b
to
f818482
Compare
26d850a
to
f818482
Compare
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.
This all looks good to me.
eea44aa
to
7e11323
Compare
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.
Looks good, will be nice to have that connection status be useful for archive plots
ffd2d69
to
7143407
Compare
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.
Thank you, looks good!
Adds
connection_state_signal
to the Archiver Plugin and makes use of it in theArchivePlotCurveItem
. More details given below:Archiver Plugin will emit to the connection_state_signal when
a. The Archiver URL doesn't exist ( --> False)
b. The network request returns an error and/or data that isn't JSON formatted ( --> False)
c. The network request returns no error and data in JSON format ( --> True)
TimePlotCurveItem
change:a. Create a channel connection signal that is emitted in
connectionStateChanged
b. In address setter, disconnect channel (if one exists) and connect the new channel
ArchiverPlotCurveItem
changes:a. Create a channel connection signal that is directly tied to the archive_channel connection signal
b. In address setter, disconnect archive_channel (if one exists) and connect the new archive_channel
c. In the address setter, prompt the PyDMArchivePlot to request archive data (needs to be done through the plot to get the time range)
In
PyDMArchiveTimePlot
, fix an issue with the default timespan not being set properly inupdateXAxis