-
Notifications
You must be signed in to change notification settings - Fork 38
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Change README logo if in dark mode (#95) * Start transform tests & minor `RandomTimeShift` optimization (#94) * Start transform tests & slight optimization in RandomTimeShift * Add new length check for TimeCrop tests * Code style * Fixed name mangling. * QAM/PSK Pulse shaping filter transition bandwidth corrected (#98) * excess bandwidth is defined in porportion to signal bandwidth, not sampling rate, thus needs to be scaled by the samples per symbol * filling in a comment to describe modification to code * QAM/PSK Pulse shaping filter transition bandwidth corrected (#98) * excess bandwidth is defined in porportion to signal bandwidth, not sampling rate, thus needs to be scaled by the samples per symbol * filling in a comment to describe modification to code * OFDM Modulator filter lengths estimated and bandwidth randomized (#99) * * cutoff frequency for LPF now randomized when using 'rand_lpf' * derives a transition bandwidth from the cutoff frequency * uses filter length approximating function for the randomized LPF * using filter estimation function for pre-computed LPF taps * Tests for visual inspection. (#103) * 91 create generation performance benchmarks for each modulation type (#104) * Initial benchmarking code. * Some benchmarks * Adding initial benchmarks. * Fix action. --------- * 75 examine ofdm generation for potential speedups for sig53 (#105) * Tests for visual inspection of modulation generation. (#102) * Optimizations show significant improvement in generation speed. * Nominal behavior after using scipy. * Adding initial Dockerfile (#108) * Incrementing version * Fix float issue (#111) * Initial draft of restructuring transforms (#106) * Flatten transforms to mirror torchvision/audio structure & add reprs * Address flake8 errors * Update transform imports with restructuring * Formatting. --------- * Fixing broken examples, formatting files, etc. * Adjustments for generation speed * Wideband generation working. * 45 consider compatibility with torch 20 (#115) * Seems to work * Workflow is broken * Python 3.7 not supported for Torch 2.0 * Adjusting test workflow * Extracted out a method for filter design. Put DSP-only things in utils/dsp (#116) * Benchmark and visualize wideband dataset generation. (#118) * Extracted out a method for filter design. Put DSP-only things in utils/dsp * New tests. * Migrated to pytest. (#119) * Added model instantiation tests for narrowband signals. (#120) * 85 gmskgfsk also needs faster filtering due to convolution of long signals with gaussian pulse shape (#121) * Change all references to convolution to scipy-based convolutions. * Missed one. * Add mypy workflow check and fix all mypy-found bugs (#123) * Fix mypy in target_transforms * Add mypy workflow for static type checking * Fix typo in mypy.yml workflow * Fix mypy in torchsig/transforms/functional.py * Fix mypy in utils/types.py * Fix mypy in torchsig/utils/ * Fix mypy in torchsig/transforms/ * Fix mypy in torchsig/datasets/ * Fix mypy in torchsig/models/ * Format with pyfmt * Fix isinstance(x, Callable) * Add PR template * Adjusting module (#130) * 131 configure package for pypi release (#132) * Adjusting installation * More information on pyproject.toml * More information on pyproject.toml * I guess some keywords in current documentation do not work :/. * Ignore distributable artifacts * More documentation and examples (#134) * More documentation and examples * Added script to train * Remove duplicate script * 126 create dockerfile and script for generating all versions of widebandsig53 (#135) * Added generation script. * Adding scripts * Fixing mypy issue. * 127 create test suite for visually validating transforms (#136) * Some transforms here * Most transforms included --------- Co-authored-by: lboegner <luke.boegner@gmail.com> Co-authored-by: Garrett Vanhoy <gmvanhoy@gmail.com> Co-authored-by: MattCarrickPL <120057274+MattCarrickPL@users.noreply.github.com>
- Loading branch information
1 parent
f6edab8
commit c9b1193
Showing
77 changed files
with
9,034 additions
and
8,521 deletions.
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
## Summary | ||
|
||
Describe your changes here specifying if the change is a bug fix, enhancement, new feature, etc. | ||
|
||
## Test Plan | ||
|
||
Describe how you tested and verified your changes here (changes captured in existing tests, built and ran new tests, etc.). | ||
|
||
## Before Submitting | ||
- [ ] Check mypy locally | ||
- `pip3 install mypy==1.2.0` | ||
- `mypy --ignore-missing-imports torchsig` | ||
- Address any error messages | ||
- [ ] Lint check locally | ||
- `pip3 install flake8` | ||
- `flake8 --select=E9,F63,F7,F82 torchsig` | ||
- Address any error messages | ||
- [ ] Run formatter if needed | ||
- `pip3 install git+https://github.com/GooeeIOT/pyfmt.git` | ||
- `pyfmt torchsig` | ||
- [ ] Run test suite locally | ||
- `pytest --ignore-glob=*_figures.py --ignore-glob=*_benchmark.py` | ||
- Ensure tests are successful prior to submitting PR | ||
|
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
name: mypy | ||
|
||
on: [push] | ||
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ["3.8", "3.10"] | ||
|
||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Install Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- uses: actions/cache@v3 | ||
with: | ||
path: ${{ env.pythonLocation }} | ||
key: ${{ env.pythonLocation }}-${{ hashFiles('pyproject.toml') }} | ||
- name: Build package | ||
run: | | ||
python -m pip install --upgrade pip | ||
python -m pip install . | ||
- name: Static type check with mypy | ||
run: | | ||
pip install mypy==1.2.0 | ||
mypy --ignore-missing-imports torchsig | ||
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,3 +11,4 @@ lightning_logs/ | |
*.pt | ||
*.jpg | ||
*.benchmarks/ | ||
dist/ |
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
Oops, something went wrong.