M-Labs
/
nac3
8
0
Fork 5
Code Issues 87 Pull Requests 3 Releases Wiki Activity

nac3artiq: implement attribute writeback #239

Merged
sb10q merged 1 commits from attribute-writeback into master 2022-03-26 20:13:58 +08:00
Conversation 2 Commits 1 Files Changed 6
pca006132 commented 2022-03-25 23:44:03 +08:00

We will only writeback attributes that are supported by the current RPC implementation: primitives, tuple and lists of lists... of primitives.

The changes are also pushed to artiq: 457f3c72ce

Closes #212

We will only writeback attributes that are supported by the current RPC implementation: primitives, tuple and lists of lists... of primitives. The changes are also pushed to artiq: https://github.com/m-labs/artiq/commit/457f3c72ce2edc33c1ea4dd8dc263a291360429b Closes #212
pca006132 commented 2022-03-25 23:47:24 +08:00
Poster

Instead of sending multiple requests, I chose to construct a header parameter that tells the host all attributes that has to be updated. Hopefully this can be more performant when we have objects with a lot of attributes.

This will send all global variable data back to the host, instead of just attributes of the experiment.

Instead of sending multiple requests, I chose to construct a header parameter that tells the host all attributes that has to be updated. Hopefully this can be more performant when we have objects with a lot of attributes. This will send all global variable data back to the host, instead of just attributes of the experiment.
pca006132 force-pushed attribute-writeback from a1bb809b7f to fef2f996fa 2022-03-26 19:15:33 +08:00 Compare
pca006132 force-pushed attribute-writeback from fef2f996fa to c955927eda 2022-03-26 19:22:09 +08:00 Compare
pca006132 commented 2022-03-26 19:23:15 +08:00
Poster

Done, no need for JSON now

Done, no need for JSON now
sb10q merged commit bf067e2481 into master 2022-03-26 20:13:58 +08:00
sb10q deleted branch attribute-writeback 2022-03-26 20:13:58 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
UROP0622
   
future-extension
   
high-priority
   
low-priority
No Label
UROP0622
future-extension
high-priority
low-priority
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/nac3#239
There is no content yet.
Delete Branch "attribute-writeback"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?