Skip to content

ENH: (CLIENTARCHIVER) remove offline_data option and replace it with beamline_data - #1260

Open
xresende wants to merge 2 commits into
masterfrom
clientarchiver-exchange-offline-beamline
Open

ENH: (CLIENTARCHIVER) remove offline_data option and replace it with beamline_data#1260
xresende wants to merge 2 commits into
masterfrom
clientarchiver-exchange-offline-beamline

Conversation

@xresende

@xresende xresende commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

The new machine archiver appliance now access all data history, rendering deprecated access to "offline" data of previous years. In its place this PR adds the option to access PVs data from the beamlines archiver appliance.

... still broken! Example:

c = ClientArchiver()
c.switch_to_beamline_data()
c.connected

fixed! It seems the problem is that when I tried this code at the firsr time the bline archiver had not implemented the secure interface yet ("https://"). Now it is working...

@fernandohds564

Copy link
Copy Markdown
Contributor

@xresende, what do you mean by still broken?

@xresende

Copy link
Copy Markdown
Contributor Author

@xresende, what do you mean by still broken?

in my computer communication with the beamline archiver using the class (which works with the machine archiver!) is broken. i started this PR but did not have time to investigate why it is broken. it is probably due to access permissions. but it can also be due to a difference is API since the BLine archiver instance is a newer version.

@xresende
xresende marked this pull request as ready for review August 13, 2026 16:38
def switch_to_machine_data(self):
"""Switch to machine data.

Sets server URL to online data URL and logs out if needed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Sets server URL to online data URL and logs out if needed.
Sets server URL to machine data URL and logs out if needed.

@VitorSouzaLNLS VitorSouzaLNLS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Matheus' suggestion.

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.

4 participants