PickleRick/DESD
1
0
Fork 0
Code Issues 2 Pull Requests Releases Activity

Lab 2: Review Notes / Potential Issues #4

New Issue
Open
opened 2025-09-01 17:47:21 +02:00 by PickleRick · 0 comments
PickleRick commented 2025-09-01 17:47:21 +02:00
Owner
Copy Link

  • Loopback mode: Sending an empty packet (FFF1) causes the module to temporarily store H and F and prepend them to the next packet header → possible pack/depack issue.

  • Packetizer: Implementation is complex; exact error not identified.

  • Color conversion (C2G): Divisor of 3 is used with an unclear approximation. -> Add comments explaining

  • Convolution: Uses unconstrained integers.

  • Code style: VHDL generally correct but complex; consider simplifying for readability.

- **Loopback mode**: Sending an empty packet (FFF1) causes the module to temporarily store H and F and prepend them to the next packet header → possible pack/depack issue. - **Packetizer**: Implementation is complex; exact error not identified. - **Color conversion (C2G)**: Divisor of 3 is used with an unclear approximation. -> Add comments explaining - **Convolution**: Uses unconstrained integers. - **Code style**: VHDL generally correct but complex; consider simplifying for readability.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
PickleRick
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: PickleRick/DESD#4
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?