delete old version of diagram, as unnecessary

unitaryfund / mitiq

Mitiq is an open source toolkit for implementing error mitigation techniques on most current intermediate-scale quantum computers.

https://mitiq.readthedocs.io

GNU General Public License v3.0

344 stars 145 forks source link

delete old version of diagram, as unnecessary #2364

Closed jordandsullivan closed 1 month ago

jordandsullivan commented 1 month ago

Description

In original branch to fix the name of the qse module in the diagram, the old version of the image was left in unnecessarily. Removing here.

License

[x] I license this contribution under the terms of the GNU GPL, version 3 and grant Unitary Fund the right to provide additional permissions as described in section 7 of the GNU GPL, version 3.

Before opening the PR, please ensure you have completed the following where appropriate.

[ ] I added unit tests for new code.
[ ] I used type hints in function signatures.
[ ] I used Google-style docstrings for functions.
[ ] I updated the documentation where relevant.
[ ] Added myself / the copyright holder to the AUTHORS file

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.22%. Comparing base (2a41ad0) to head (732e12f).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #2364 +/- ## ======================================= Coverage 98.22% 98.22% ======================================= Files 87 87 Lines 4059 4059 ======================================= Hits 3987 3987 Misses 72 72 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jordandsullivan commented 1 month ago

Should we rename the diagram-v2 back to "diagram" in addition to deleting the old file?

I was reminded of this comic :)

Lol I don't have a strong preference on this. But tbf it's not like we're calling it "final version," just incrementing the version number so it's clear a previous version existed in case people still have it referenced somewhere.