68 move samples per bit function out of ibis ami tester notebook#80
Open
capn-freako wants to merge 43 commits into
Open
68 move samples per bit function out of ibis ami tester notebook#80capn-freako wants to merge 43 commits into
capn-freako wants to merge 43 commits into
Conversation
…ration for adding a new testing tool.
==> This did NOT fix the performance problem.
…-out-of-ibis_ami_tester-notebook' into 68-move-samples_per_bit-function-out-of-ibis_ami_tester-notebook
- Needs some additional plot rearrangement.
- Still needs linting!
- Next step is to add linearity check.
- Factored testing code in `ami_tests.py`.
- Run time blow-up w/ larger `nbits`. - Long tail of zeros in test input.
- New abstract class for test helpers. - New dataclasses for better type checking. - Model response plotting cleaned up and generalized. - DFE adaptation plotting added. - Bugs in IBIS model EmPy master template fixed. - Cleaned up GetWave() response fetching. - Added profiling output and test run configuration files to `.gitignore`.
- Includes absorbing channel response into test run defs. - Also, absorbs bit_time, sample_interval, and nbits.
- Restored linearity checking. - Gave individual testers ownership over figure size.
added 13 commits
May 8, 2026 08:51
- Removed `is_tx` flag in several places. - Several changes to function call signatures.
- validate `AMI_Init()` and `AMI_GetWave()` usage upon instantiation, - provide abstract `@property` interfaces to all attributes, and - run its `ami_tst()` function when called. Changed implementation of test helpers from `Protocol` to subclass. Eliminated pesky warnings from `plt.legend()`.
…is_ami_tester-notebook
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This turned into a complete revamping of the IBIS-AMI model testing infrastructure.