[go: nahoru, domu]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In-repository performance benchmarks and user interface for rate adaption #1

Merged
merged 26 commits into from
Sep 6, 2021

Conversation

XQP-Munich
Copy link
Collaborator

The repository is currently still missing the LDPC codes and the Julia scripts used to pre-process them for use in the C++ binaries (reading alist files, storing in sparse column format, converting the codes to static data, converting the rate adaption).

Currently, improved LDPC constructions are still running. The LDPC codes with the best performance will be added together with the most recent Julia scripts when the constructions are complete.

@XQP-Munich XQP-Munich self-assigned this Jul 6, 2021
@XQP-Munich XQP-Munich marked this pull request as draft July 6, 2021 16:13
@XQP-Munich XQP-Munich force-pushed the feature/pre-release branch 2 times, most recently from a125c9d to 7c03536 Compare July 7, 2021 13:54
@XQP-Munich XQP-Munich marked this pull request as ready for review July 7, 2021 13:56
@XQP-Munich XQP-Munich closed this Jul 7, 2021
@XQP-Munich
Copy link
Collaborator Author

Some commits in this branch were used to make the main branch have passing unit tests.
The goals of this pull request remain the same and will be addressed soon.

@XQP-Munich XQP-Munich reopened this Jul 7, 2021
@XQP-Munich XQP-Munich changed the title Include LDPC codes and Julia processing scripts In-repository performance benchmarks and user interface for rate adaption Aug 24, 2021
@XQP-Munich
Copy link
Collaborator Author

The LDPC codes and aff3ct simulation results have been added to master. This branch has been rebased, since those changes are required for completing the goals of this PR.

The remaining goals include in-repository performance benchmarks, as well as a fully functional user interface for the rate adaption.

@XQP-Munich
Copy link
Collaborator Author

The current state does include a complete implementation of rate adaptive decoding with passing tests. However, extensive (let alone automatic) performance benchmarks are NOT included. Thus, only one half of the original aims of this PR are completed. Nevertheless, due to the large amount of commits and prolonged preliminary status of the main branch, the PR is merged. The remaining work on performance benchmarks will be completed in further PRs.

There also seems to be a discrepancy between the performance of the provided BP-decoder and the results obtained using AFF3CT. This will be adressed in an issue.

@XQP-Munich XQP-Munich merged commit dc30759 into main Sep 6, 2021
@XQP-Munich XQP-Munich deleted the feature/pre-release branch September 6, 2021 14:57
@XQP-Munich XQP-Munich removed their assignment Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant