Title: Deprecate/remove the action-flag · Issue #43 · SoftwareQuTech/CQC-Python · GitHub
Open Graph Title: Deprecate/remove the action-flag · Issue #43 · SoftwareQuTech/CQC-Python
X Title: Deprecate/remove the action-flag · Issue #43 · SoftwareQuTech/CQC-Python
Description: As far as I know, the only functionality of the action flag is that it lets the backend process command headers recursively. This is also visible in the lines below: on line 311 of the removed code, the method is recursively invoked. Thi...
Open Graph Description: As far as I know, the only functionality of the action flag is that it lets the backend process command headers recursively. This is also visible in the lines below: on line 311 of the removed code...
X Description: As far as I know, the only functionality of the action flag is that it lets the backend process command headers recursively. This is also visible in the lines below: on line 311 of the removed code...
Opengraph URL: https://github.com/SoftwareQuTech/CQC-Python/issues/43
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Deprecate/remove the action-flag","articleBody":"As far as I know, the only functionality of the `action` flag is that it lets the backend process command headers recursively. This is also visible in the lines below: on line 311 of the removed code, the method is recursively invoked. This adds unnecessary complexity for no good reason, in my opinion. Without the action flag, it is still possible to send multiple commands at once. For instance, consider the following CQC program (in pseudo-CQC):\r\n\r\n```\r\nCQC Header. Type = CQCType.COMMAND. Length = 3 command headers\r\nCommand Header. Instruction = X. Action = False\r\nCommand Header. Instruction = Y. Action = False\r\nCommand Header. Instruction = Z. Action = False\r\n```\r\n\r\nEven though `action` is set to `False`, all three commands will be executed sequentially. This is my understanding of sequences and the `action` flag. Am I missing something?\r\n\r\nRegarding the trouble with the logic: During the design phase of my project, I consulted Leon to discuss some ideas and possible solutions. At that point, I had some questions regarding sequences. All these questions were answered and there are no problems anymore. In my final implementation, the sequence functionality works perfectly (albeit without the action flag, and without sequence headers).\r\n\r\n_Originally posted by @VictorPrins in https://github.com/SoftwareQuTech/CQC-Python/pull/35_","author":{"url":"https://github.com/AckslD","@type":"Person","name":"AckslD"},"datePublished":"2019-10-23T09:03:40.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/43/CQC-Python/issues/43"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:29660f5c-97dd-4a18-9675-357aa8508bfd |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 8CE8:3FE34A:134ADC9:1984D30:69819B7E |
| html-safe-nonce | 565b6ebc698850a0456916575e1196cf32e98b7cf39f061d9b008761bb301c04 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4Q0U4OjNGRTM0QToxMzRBREM5OjE5ODREMzA6Njk4MTlCN0UiLCJ2aXNpdG9yX2lkIjoiNDcwNzM3ODkzOTUyNDM4OTc1OCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | f1ae16e70afc1bb331e8b522e1c3cf82a4b763b224331af8275c54224b139e60 |
| hovercard-subject-tag | issue:511180947 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/SoftwareQuTech/CQC-Python/43/issue_layout |
| twitter:image | https://opengraph.githubassets.com/faf1f847bca9b44d6f3d7326df7688c23d88d7d09696ca700a6ed3b1ac76dcde/SoftwareQuTech/CQC-Python/issues/43 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/faf1f847bca9b44d6f3d7326df7688c23d88d7d09696ca700a6ed3b1ac76dcde/SoftwareQuTech/CQC-Python/issues/43 |
| og:image:alt | As far as I know, the only functionality of the action flag is that it lets the backend process command headers recursively. This is also visible in the lines below: on line 311 of the removed code... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | AckslD |
| hostname | github.com |
| expected-hostname | github.com |
| None | e137814e266030874fd2c86863529d0622b13889eeda04148c57654b6ea84ad6 |
| turbo-cache-control | no-preview |
| go-import | github.com/SoftwareQuTech/CQC-Python git https://github.com/SoftwareQuTech/CQC-Python.git |
| octolytics-dimension-user_id | 43809204 |
| octolytics-dimension-user_login | SoftwareQuTech |
| octolytics-dimension-repository_id | 177956316 |
| octolytics-dimension-repository_nwo | SoftwareQuTech/CQC-Python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 177956316 |
| octolytics-dimension-repository_network_root_nwo | SoftwareQuTech/CQC-Python |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | dd58d68a7813bbec9c91422c8c35f4af33832d70 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width