search

larpix / larpix-control

Control the LArPix chip
Other
4 stars 12 forks source link

Add write to tx registers in init_network, autopep8 #248

Closed peter-madigan closed 3 years ago

mualem commented 3 years ago

I think this is part of it. This should make it more reliable to establish communication with the differential readout. I think the more fundamental problem I'm seeing is that I'm not able to configure with a chip_id!=1, and if I run an init_network it seems to derail a working system, even if I'm not trying to change the chip_id.

mualem commented 3 years ago

I made some tests with it, since you were so kind to supply the quick fixes. I'm going to have to head out now though.

peter-madigan commented 3 years ago

No problem. Does this happen when using the default values in the script or with a provided configuration file? If it's when using a configuration file, can you send me what you are using? I'd like to see if it is just something simple.

On Fri, Oct 8, 2021 at 4:03 PM Leon Mualem @.***> wrote:

I made some tests with it, since you were so kind to supply the quick fixes. I'm going to have to head out now though.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/larpix/larpix-control/pull/248#issuecomment-939158699, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEKVRQCB54TZO6X3QOC2BBLUF52F3ANCNFSM5FUQUIBA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

mualem commented 3 years ago

It happens with the defaults in the script.

Leon Mualem Lauritsen Lab MC 356-48 Caltech 1200 E. California Blvd. Pasadena, CA 91125 1 (626) 817-3459

From: peter-madigan @.> Sent: Friday, October 8, 2021 4:11 PM To: larpix/larpix-control @.> Cc: Mualem, Leon @.>; Comment @.> Subject: Re: [larpix/larpix-control] Add write to tx registers in init_network, autopep8 (#248)

No problem. Does this happen when using the default values in the script or with a provided configuration file? If it's when using a configuration file, can you send me what you are using? I'd like to see if it is just something simple.

On Fri, Oct 8, 2021 at 4:03 PM Leon Mualem @.***> wrote:

I made some tests with it, since you were so kind to supply the quick fixes. I'm going to have to head out now though.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/larpix/larpix-control/pull/248#issuecomment-939158699, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEKVRQCB54TZO6X3QOC2BBLUF52F3ANCNFSM5FUQUIBA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/larpix/larpix-control/pull/248#issuecomment-939160893, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEN5E72FNUIIT7FVUQDISDLUF53APANCNFSM5FUQUIBA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.