feat: bugfix_single_subplot_pulse_diagram
Explanation of changes
This MR implements various improvements regarding plotting.
-
Performance : time for plotting was extremely long. Only one plot per port now is used. In linear signals (ramp and square pulses) the number of points used to plot the signal is reduced to the bare minimum
-
Repetition : Discarded all the useless points that were created a zeros in the plot due to repetition. Timing show only the first repetition
-
Offset : Now pulse are properly displayed including offsets and waveforms
-
New functions have been added
Motivation of changes
Increased performance and memory efficiency.
Merge checklist
See also merge request guidelines
-
Merge request has been reviewed (in-depth by a knowledgeable contributor), and is approved by a project maintainer. -
New code is covered by unit tests (or N/A). -
New code is documented and docstrings use numpydoc format (or N/A). -
New functionality: considered making private instead of extending public API (or N/A). -
Public API changed: added @deprecatedand entry in deprecated code suggestions (or N/A). -
Newly added/adjusted documentation and docstrings render properly (or N/A). -
Pipeline fix or dependency update: post in #software-for-developerschannel to mergemainback in or update local packages (or N/A). -
Tested on hardware (or N/A). -
CHANGELOG.mdfor breaking changes andAUTHORS.mdhave been updated (or N/A). -
Update Hardware backends documentation if backend interface change or N/A -
Check whether performance is significantly affected by looking at the Performance metrics results. -
Windows tests in CI pipeline pass (manually triggered by maintainers before merging). - Maintainers do not hit Auto-merge, we need to actively check as manual tests do not block pipeline
For reference, the issues workflow is described in the contribution guidelines.
Edited by Christos Samouchos