Skip to content
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

Allow to define specific HTTP and ZMQ ports #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tiagocoutinho
Copy link

Hi,

"Soon" ALBA will have an Eiger detector.
Right now, we started making some tests using an eiger simulator using a sample dectris dataset.
We fire up the simulator using configurable http port (default 8000) and zmq port (default 9999) to avoid the need of admin privileges for launching on port 80.

This PR makes it possible to create a Lima Eiger Camera object with optional http port and zmq port parameters so it can connect to a simulator for example.

@tiagocoutinho tiagocoutinho changed the title Allow to define specific HTTP and ZMQ ports WIP: Allow to define specific HTTP and ZMQ ports Feb 26, 2020
@tiagocoutinho tiagocoutinho changed the title WIP: Allow to define specific HTTP and ZMQ ports Allow to define specific HTTP and ZMQ ports Mar 5, 2020
@tiagocoutinho
Copy link
Author

Should be working now.
@laurent-claustre or @alejandrohomsp could you have a quick look at this one?
Thanks a lot!

@laurent-claustre
Copy link

Hi @tiagocoutinho, we are super busy with the restart after one and 1/2 half of shutdown. We also found a bug in this plugin running Eiger1 detector for the IntTriggerMulti and ExtTriggerMulti and ExtGate.
Which detector are you using, Eiger1 or Eiger2?

@tiagocoutinho
Copy link
Author

Hi @laurent-claustre,

No rush from my side.

We are simulating an eiger2 based on simplon 1.6 API (document version 1.3.2) which is what we will have on the beamline.
So far we just simulate IntTriggerSingle ("ints" trigger mode for the eiger) because we are interested in studying the flux of data to help us design the infrastructure.

Could you tell me what kind of machine you have running Lima for the eiger2 (cpus, ram)?
It would also help to know the lima configuration (nb processing tasks, and nb saving tasks, max buffer size)

Best of luck with the restart!

@laurent-claustre
Copy link

Hi Tiago,
this is our new configuration for the Eiger2 Lima PC, a PowerEdge R740XD Server with:

  • 1 PowerEdge R740/R740XD Carte mère
  • 2 Intel Xeon Silver 4216 (2,1GHz, 16C/32T, cache 22Mo, 9,6GT/s, 100W, Turbo, HT), mémoire
    DDR4 2400MHz
  • 1 2933MT/s RDIMMs
  • 12 16Go RDIMM, 2933MT/s, Double rangé
  • 2 Disque dur Hot Plug 1To NL-SAS 12Gbit/s 512n 7200tr/min 2,5"
  • 2 Dell Carte d'extension 375Go, NVMe, Express Flash hautes performances, mi-hauteur milongueur,
  • 2 Mellanox ConnectX-5 Single Port EDR VPI QSFP28 100Gb PCIe Adapter, Full Height
  • 1 Intel X710 Deux ports 10Gb Connexion directe/SFP+ et I350 Deux ports 1GbE Carte fille réseau

This is a new beast !! around 11keuros HT

@tiagocoutinho
Copy link
Author

@laurent-claustre, thanks a lot for the info.

@laurent-claustre
Copy link

Hi Tiago, sorry for the late merge, we review the code today and merge it this week. We have now a detector running (500K) to test your changes.

@tiagocoutinho
Copy link
Author

Hi, no worries on my side. I am not in a hurry.
Thanks for having a look.

@laurent-claustre
Copy link

laurent-claustre commented May 4, 2020

Hi Tiago, we are pretty ready to merge and to delivery a version 1.9.4. Question, do you use the LimaCCDs pytango server?
If yes I suppose you will need new properties to be able to change the http and stream port numbers?
So we can prepare a new package for the tango camer plugin Eiger.

@tiagocoutinho
Copy link
Author

Hi @laurent-claustre,
For my tests I don't use tango. I am using a lima-cli I wrote for the purpose (which I will make public soon :-) so I don't need the extra properties in the lima tango eiger myself.
I might soon use it so I can make a PR with the changes in the Lima tango eiger repo.

@laurent-claustre
Copy link

Hi @tiagocoutinho , thanks. no more PR needed, we added the new properties to the tango plugin.
Do you plan to use conda packages at ALBA?

@tiagocoutinho
Copy link
Author

Ah, ok great!
For Lima yes for sure we use conda. The builds on the esrf-bcu channel are extremely useful.

@laurent-claustre
Copy link

Hi tiago, new package 1.9.4 is available, sorry for the conflict with your PR, I did change the .sip file.
Could your mark it as resolved once tested.
Thanks.

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.

2 participants