Skip to content

Conversation

@bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented Apr 25, 2025

What does this Pull Request accomplish?

Enable the == and != operators to compare waveforms by value.

Enable the str() and repr() functions to display a text representation of a waveform.

Rearrange the AnalogWaveform constructor parameters to match the repr() function output and allow specifying extended properties, timing, and scaling at construction time.

Why should this Pull Request be merged?

Closes AB#3097256 - Support eq/repr

What testing has been done?

Ran unit tests

@bkeryan bkeryan requested a review from csjall as a code owner April 25, 2025 23:10
@bkeryan bkeryan force-pushed the users/bkeryan/append-waveform branch from 53aaf60 to 6934f10 Compare April 26, 2025 16:00
@bkeryan bkeryan force-pushed the users/bkeryan/waveform-magic branch from cf223c9 to 66b0af6 Compare April 26, 2025 16:16
@bkeryan bkeryan force-pushed the users/bkeryan/append-waveform branch from 3caa239 to 18a9000 Compare April 26, 2025 16:30
@bkeryan bkeryan force-pushed the users/bkeryan/waveform-magic branch 2 times, most recently from 65b8cca to 245b6b3 Compare April 28, 2025 22:32
@bkeryan bkeryan force-pushed the users/bkeryan/append-waveform branch from bead0e1 to 2d6664c Compare April 28, 2025 23:48
@bkeryan bkeryan force-pushed the users/bkeryan/waveform-magic branch from 245b6b3 to 6569aa6 Compare April 28, 2025 23:50
Base automatically changed from users/bkeryan/append-waveform to main April 29, 2025 21:41
@bkeryan bkeryan requested a review from jfriedri-ni April 29, 2025 21:41
@bkeryan bkeryan force-pushed the users/bkeryan/waveform-magic branch from 6569aa6 to 0807d90 Compare April 29, 2025 21:44
@github-actions
Copy link
Contributor

github-actions bot commented Apr 29, 2025

Test Results

   10 files  ±  0     10 suites  ±0   23s ⏱️ -1s
  391 tests + 41    386 ✅ + 42   5 💤  -  1  0 ❌ ±0 
3 910 runs  +410  3 860 ✅ +420  50 💤  - 10  0 ❌ ±0 

Results for commit fff1d57. ± Comparison against base commit 0d515bc.

♻️ This comment has been updated with latest results.

@bkeryan bkeryan merged commit 4ae37e4 into main Apr 30, 2025
13 checks passed
@bkeryan bkeryan deleted the users/bkeryan/waveform-magic branch April 30, 2025 15:48
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.

3 participants