Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@zdomke
Copy link
Contributor

@zdomke zdomke commented Aug 26, 2024

Adds connection_state_signal to the Archiver Plugin and makes use of it in the ArchivePlotCurveItem. More details given below:

  1. 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)

  2. 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

  3. 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)

  4. In PyDMArchiveTimePlot, fix an issue with the default timespan not being set properly in updateXAxis

@zdomke zdomke force-pushed the dev_invalid_archive_channel branch from 4db392b to f818482 Compare August 26, 2024 23:14
@zdomke zdomke marked this pull request as ready for review August 26, 2024 23:25
@zdomke zdomke force-pushed the dev_invalid_archive_channel branch from 26d850a to f818482 Compare August 28, 2024 18:30
Copy link
Collaborator

@YektaY YektaY left a 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.

@zdomke zdomke force-pushed the dev_invalid_archive_channel branch from eea44aa to 7e11323 Compare September 4, 2024 23:41
Copy link
Collaborator

@jbellister-slac jbellister-slac left a 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

zdomke added a commit to zdomke/pydm that referenced this pull request Sep 13, 2024
@zdomke zdomke force-pushed the dev_invalid_archive_channel branch from ffd2d69 to 7143407 Compare September 13, 2024 23:22
Copy link
Collaborator

@jbellister-slac jbellister-slac left a 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!

@jbellister-slac jbellister-slac merged commit 81d9b6b into slaclab:master Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants