| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:4c15f23c-a896-f7d9-4a4a-28d82b1bce1d |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9794:3C6F7B:60E27D1:81B78B9:6A5DF194 |
| html-safe-nonce | 3b85c8c2b0ce12823a8f83581d21c6af37b0d869906d2443539b3d9e043cb3b6 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5Nzk0OjNDNkY3Qjo2MEUyN0QxOjgxQjc4Qjk6NkE1REYxOTQiLCJ2aXNpdG9yX2lkIjoiNDc4Mzg4MTYyMjg2NDY1NDc0MCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | d81634ee9ac90e09a101450dfdcbf8e0a6f18a581b0fbddde1712805f53dd63a |
| hovercard-subject-tag | pull_request:2022399691 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/canopen-python/canopen/538/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/55298aba0103db462f20ec9e8b5db37fb1c62e3aefc92ce784f2da41c7c0181b/canopen-python/canopen/pull/538 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/55298aba0103db462f20ec9e8b5db37fb1c62e3aefc92ce784f2da41c7c0181b/canopen-python/canopen/pull/538 |
| og:image:alt | The count entry itself is not part of the data (according to CiA 301), thus should not be yielded from an iterator. That matches the behavior of SdoArray, which also yields only the array contents... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | acolomb |
| hostname | github.com |
| expected-hostname | github.com |
| None | 7f8603048a7a8270fc5b845c28ec71e80a9a899268387bcf65814ee1b51c0cc4 |
| turbo-cache-control | no-cache |
| go-import | github.com/canopen-python/canopen git https://github.com/canopen-python/canopen.git |
| octolytics-dimension-user_id | 200581454 |
| octolytics-dimension-user_login | canopen-python |
| octolytics-dimension-repository_id | 68737600 |
| octolytics-dimension-repository_nwo | canopen-python/canopen |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 68737600 |
| octolytics-dimension-repository_network_root_nwo | canopen-python/canopen |
| 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 | e2a5561aeb049d0632a92906af784af5d414c603 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/canopen-python/canopen/pull/538#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcanopen-python%2Fcanopen%2Fpull%2F538 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub Copilot appDirect agents from issue to merge | https://github.com/features/ai/github-app |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| View all resources | https://github.com/resources |
| GitHub SponsorsFund open source developers | https://github.com/open-source/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/open-source/accelerator |
| GitHub Stars | https://stars.github.com |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/enterprise/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcanopen-python%2Fcanopen%2Fpull%2F538 |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=canopen-python%2Fcanopen |
| Reload | https://github.com/canopen-python/canopen/pull/538 |
| Reload | https://github.com/canopen-python/canopen/pull/538 |
| Reload | https://github.com/canopen-python/canopen/pull/538 |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
|
canopen-python
| https://github.com/canopen-python |
| canopen | https://github.com/canopen-python/canopen |
|
Notifications
| https://github.com/login?return_to=%2Fcanopen-python%2Fcanopen |
|
Fork
359
| https://github.com/login?return_to=%2Fcanopen-python%2Fcanopen |
|
Star
560
| https://github.com/login?return_to=%2Fcanopen-python%2Fcanopen |
|
Code
| https://github.com/canopen-python/canopen |
|
Issues
53
| https://github.com/canopen-python/canopen/issues |
|
Pull requests
20
| https://github.com/canopen-python/canopen/pulls |
|
Discussions
| https://github.com/canopen-python/canopen/discussions |
|
Actions
| https://github.com/canopen-python/canopen/actions |
|
Projects
| https://github.com/canopen-python/canopen/projects |
|
Security and quality
0
| https://github.com/canopen-python/canopen/security |
|
Insights
| https://github.com/canopen-python/canopen/pulse |
|
Code
| https://github.com/canopen-python/canopen |
|
Issues
| https://github.com/canopen-python/canopen/issues |
|
Pull requests
| https://github.com/canopen-python/canopen/pulls |
|
Discussions
| https://github.com/canopen-python/canopen/discussions |
|
Actions
| https://github.com/canopen-python/canopen/actions |
|
Projects
| https://github.com/canopen-python/canopen/projects |
|
Security and quality
| https://github.com/canopen-python/canopen/security |
|
Insights
| https://github.com/canopen-python/canopen/pulse |
| acolomb | https://github.com/acolomb |
| canopen-python:master | https://github.com/canopen-python/canopen/tree/master |
| acolomb:sdorecord-skip-count | https://github.com/acolomb/canopen/tree/sdorecord-skip-count |
| Conversation | https://github.com/canopen-python/canopen/pull/538 |
| Commits11 (11) | https://github.com/canopen-python/canopen/pull/538/commits |
| Checks | https://github.com/canopen-python/canopen/pull/538/checks |
| Files changed | https://github.com/canopen-python/canopen/pull/538/files |
| Skip the "highest subindex" entry when iterating over SdoRecord | https://github.com/canopen-python/canopen/pull/538#top |
| acolomb | https://github.com/acolomb |
| canopen-python:master | https://github.com/canopen-python/canopen/tree/master |
| acolomb:sdorecord-skip-count | https://github.com/acolomb/canopen/tree/sdorecord-skip-count |
|
| https://github.com/acolomb |
| acolomb | https://github.com/acolomb |
| Aug 16, 2024 | https://github.com/canopen-python/canopen/pull/538#issue-2469624175 |
| #430 | https://github.com/canopen-python/canopen/pull/430 |
| last comment there | https://github.com/christiansandberg/canopen/pull/430#issuecomment-2292416333 |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
|
| https://github.com/acolomb |
| Skip the "highest subindex" entry when iterating over SdoRecord. | https://github.com/canopen-python/canopen/pull/538/commits/3627ed75299408387bdf43bc9cd914c2f9daf343 |
| 3627ed7 | https://github.com/canopen-python/canopen/pull/538/commits/3627ed75299408387bdf43bc9cd914c2f9daf343 |
| https://github.com/codecov-commenter |
| codecov-commenter | https://github.com/codecov-commenter |
| Aug 16, 2024 | https://github.com/canopen-python/canopen/pull/538#issuecomment-2292930725 |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| https://private-user-images.githubusercontent.com/152432831/340411587-e90313f4-9d3a-4b63-8b54-cfe14e7ec20d.svg?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3ODQ1NDE4ODksIm5iZiI6MTc4NDU0MTU4OSwicGF0aCI6Ii8xNTI0MzI4MzEvMzQwNDExNTg3LWU5MDMxM2Y0LTlkM2EtNGI2My04YjU0LWNmZTE0ZTdlYzIwZC5zdmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwNzIwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDcyMFQwOTU5NDlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00M2I0ZDFlODk5YzdmZTg4MzViMmIwMmMxNGI1NTY3MWIwZTU2NjNiYWEyMWRmM2MwYWMzMTQzMTNkZjQzZjViJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZyZXNwb25zZS1jb250ZW50LXR5cGU9aW1hZ2UlMkZzdmclMkJ4bWwifQ.2gzyaRCU0dOtZAbNa8uuYqAQIVfF3QIpAOiLzBPPWOg |
| Codecov | https://app.codecov.io/gh/christiansandberg/canopen/pull/538?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg |
| (273bab1) | https://app.codecov.io/gh/christiansandberg/canopen/commit/273bab17400f0eb14c5e9e7714fa88de3fce2e7f?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg |
| (5eb88d7) | https://app.codecov.io/gh/christiansandberg/canopen/commit/5eb88d7b3c09704d598f573e0ee31fdd17d4aa2a?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg |
| Codecov GitHub app | https://github.com/apps/codecov/installations/select_target |
| https://app.codecov.io/gh/christiansandberg/canopen/pull/538?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg |
| Files | https://app.codecov.io/gh/christiansandberg/canopen/pull/538?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg |
| canopen/sdo/base.py | https://app.codecov.io/gh/christiansandberg/canopen/pull/538?src=pr&el=tree&filepath=canopen%2Fsdo%2Fbase.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Sandberg#diff-Y2Fub3Blbi9zZG8vYmFzZS5weQ== |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| https://github.com/erlend-aasland |
| erlend-aasland | https://github.com/erlend-aasland |
|
Aug 16, 2024
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2243420659 |
|
View reviewed changes
| https://github.com/canopen-python/canopen/pull/538/files/3627ed75299408387bdf43bc9cd914c2f9daf343 |
| erlend-aasland | https://github.com/erlend-aasland |
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2243420659 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| acolomb | https://github.com/acolomb |
| August 17, 2024 19:36 | https://github.com/canopen-python/canopen/pull/538#commits-pushed-a46d114 |
|
| https://github.com/acolomb |
| Extend docs. | https://github.com/canopen-python/canopen/pull/538/commits/a46d114cca5aea276a257f5c98033e3debbe5696 |
| a46d114 | https://github.com/canopen-python/canopen/pull/538/commits/a46d114cca5aea276a257f5c98033e3debbe5696 |
|
| https://github.com/acolomb |
| Fix typos. | https://github.com/canopen-python/canopen/pull/538/commits/27b37246f9a3847bcd403cbc026d99ee61f94804 |
| 27b3724 | https://github.com/canopen-python/canopen/pull/538/commits/27b37246f9a3847bcd403cbc026d99ee61f94804 |
|
| https://github.com/acolomb |
| Remove one of the sub-objects in a record for testing. | https://github.com/canopen-python/canopen/pull/538/commits/e5d78b7503524cd2ad2a04a6c3b44cadb91fed5e |
| e5d78b7 | https://github.com/canopen-python/canopen/pull/538/commits/e5d78b7503524cd2ad2a04a6c3b44cadb91fed5e |
|
| https://github.com/acolomb |
| Flesh out the Pre-defined error field object in sample.eds. | https://github.com/canopen-python/canopen/pull/538/commits/86c3350937f07e5e6baf2dfd0784167c54f66e5d |
| 86c3350 | https://github.com/canopen-python/canopen/pull/538/commits/86c3350937f07e5e6baf2dfd0784167c54f66e5d |
|
| https://github.com/acolomb |
| Add test for SdoArray dynamically generated member variables. | https://github.com/canopen-python/canopen/pull/538/commits/daf8ca68f5712673b03a7cb6b35e138a8424a1ee |
| daf8ca6 | https://github.com/canopen-python/canopen/pull/538/commits/daf8ca68f5712673b03a7cb6b35e138a8424a1ee |
|
| https://github.com/acolomb |
| Add test for SdoArray length and iteration count. | https://github.com/canopen-python/canopen/pull/538/commits/5cbb00ef9129643be462ef5feab271e701947802 |
| 5cbb00e | https://github.com/canopen-python/canopen/pull/538/commits/5cbb00ef9129643be462ef5feab271e701947802 |
|
| https://github.com/acolomb |
| Add test for SdoRecord length and iteration count. | https://github.com/canopen-python/canopen/pull/538/commits/30c407127b12bcdfc7afce2449aa3811e8da3e0c |
| 30c4071 | https://github.com/canopen-python/canopen/pull/538/commits/30c407127b12bcdfc7afce2449aa3811e8da3e0c |
| https://github.com/acolomb |
| acolomb | https://github.com/acolomb |
|
Aug 17, 2024
| https://github.com/canopen-python/canopen/pull/538#ref-pullrequest-2471639200 |
|
Tests: Verify some basic SDO record and array assumptions
#539
| https://github.com/canopen-python/canopen/pull/539 |
|
| https://github.com/acolomb |
| Merge branch 'test-sdo-variables' into sdorecord-skip-count | https://github.com/canopen-python/canopen/pull/538/commits/36150d7ecf41c3acc7b720611b0660ab9f4cef4f |
| 36150d7 | https://github.com/canopen-python/canopen/pull/538/commits/36150d7ecf41c3acc7b720611b0660ab9f4cef4f |
| https://github.com/acolomb |
| acolomb | https://github.com/acolomb |
| Aug 17, 2024 | https://github.com/canopen-python/canopen/pull/538#issuecomment-2294951654 |
| #539 | https://github.com/canopen-python/canopen/pull/539 |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| https://github.com/erlend-aasland |
| erlend-aasland | https://github.com/erlend-aasland |
|
Aug 17, 2024
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2244150468 |
|
View reviewed changes
| https://github.com/canopen-python/canopen/pull/538/files/36150d7ecf41c3acc7b720611b0660ab9f4cef4f |
| erlend-aasland | https://github.com/erlend-aasland |
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2244150468 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| acolomb | https://github.com/acolomb |
| August 17, 2024 23:01 | https://github.com/canopen-python/canopen/pull/538#commits-pushed-8b7ba4c |
|
| https://github.com/acolomb |
| Merge branch 'master' into sdorecord-skip-count | https://github.com/canopen-python/canopen/pull/538/commits/8b7ba4ce693de79dece79108ec67ae5779cd97b1 |
| 8b7ba4c | https://github.com/canopen-python/canopen/pull/538/commits/8b7ba4ce693de79dece79108ec67ae5779cd97b1 |
|
| https://github.com/acolomb |
| No need to expect failure anymore. | https://github.com/canopen-python/canopen/pull/538/commits/5eb88d7b3c09704d598f573e0ee31fdd17d4aa2a |
| 5eb88d7 | https://github.com/canopen-python/canopen/pull/538/commits/5eb88d7b3c09704d598f573e0ee31fdd17d4aa2a |
| https://github.com/erlend-aasland |
| erlend-aasland | https://github.com/erlend-aasland |
|
Aug 17, 2024
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2244151075 |
|
View reviewed changes
| https://github.com/canopen-python/canopen/pull/538/files/5eb88d7b3c09704d598f573e0ee31fdd17d4aa2a |
| erlend-aasland | https://github.com/erlend-aasland |
| https://github.com/canopen-python/canopen/pull/538#pullrequestreview-2244151075 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
| https://github.com/acolomb |
| acolomb | https://github.com/acolomb |
| 6bc90a8 | https://github.com/canopen-python/canopen/commit/6bc90a817f049b52fbeb1201c699e90ca36a6136 |
| Aug 17, 2024 | https://github.com/canopen-python/canopen/pull/538#event-13920583359 |
| https://github.com/acolomb |
| acolomb | https://github.com/acolomb |
| August 17, 2024 21:05 | https://github.com/canopen-python/canopen/pull/538#event-13920584312 |
| https://github.com/sveinse |
| sveinse | https://github.com/sveinse |
|
Jun 22, 2025
| https://github.com/canopen-python/canopen/pull/538#ref-pullrequest-2295641164 |
|
Making SdoArray iterator behave equal to ODArray and *Record
#430
| https://github.com/canopen-python/canopen/pull/430 |
| https://github.co/hiddenchars |
| https://github.com/canopen-python/canopen/pull/{{ revealButtonHref }} |
| Sign up for free | https://github.com/join?source=comment-repo |
| Sign in to comment | https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcanopen-python%2Fcanopen%2Fpull%2F538 |
|
| https://github.com/erlend-aasland |
|
erlend-aasland
| https://github.com/erlend-aasland |
|
| https://github.com/canopen-python/canopen/pull/538/files/5eb88d7b3c09704d598f573e0ee31fdd17d4aa2a |
| Please reload this page | https://github.com/canopen-python/canopen/pull/538 |
|
| https://github.com/acolomb |
|
| https://github.com/codecov-commenter |
|
| https://github.com/erlend-aasland |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |