Skip to content

Add support for nlayers > 1 in nrRateMatchLDPC - #81

Merged
catkira merged 1 commit into
catkira:masterfrom
fredsonnenwald:ldpc_nlayers
May 14, 2026
Merged

catkira merged 1 commit into
catkira:masterfrom
fredsonnenwald:ldpc_nlayers

Conversation

@fredsonnenwald

Copy link
Copy Markdown
Contributor

This PR adds support for up to four layers in the LDPC rate matching code. The changes mostly consist of making sure that _rateMatch treats d as a 2D array. I've updated the MATLAB test to check that the correct result is produced.

$ python -m pytest tests/matlab/test_nrRateMatchLDPC_matlab.py
============================= test session starts ==============================
platform linux -- Python 3.12.3, pytest-9.0.3, pluggy-1.6.0
rootdir: /home/user/py3gpp
configfile: pyproject.toml
collected 576 items                                                            

tests/matlab/test_nrRateMatchLDPC_matlab.py ............................ [  4%]
........................................................................ [ 17%]
........................................................................ [ 29%]
........................................................................ [ 42%]
........................................................................ [ 54%]
........................................................................ [ 67%]
........................................................................ [ 79%]
........................................................................ [ 92%]
............................................                             [100%]

============================= 576 passed in 18.27s =============================

@catkira catkira added the enhancement New feature or request label May 14, 2026
@catkira

catkira commented May 14, 2026

Copy link
Copy Markdown
Owner

@fredsonnenwald

Copy link
Copy Markdown
Contributor Author

Sorry @catkira, I thought I had pushed that fix already! Hopefully that does it now.

@catkira
catkira merged commit 68abfb5 into catkira:master May 14, 2026
5 checks passed
@fredsonnenwald
fredsonnenwald deleted the ldpc_nlayers branch July 3, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants