feat: Remove unused rewrite arithmetic logic, deprecate recalculation_table

[MarquessV]

Description

Closes #1786

Checklist

• The PR targets the master branch
• The above description motivates these changes.
• The change is atomic and can be described by a single commit (your PR will be squashed on merge).
• All changes to code are covered via unit tests.
• Parameters and return values have type hints with PEP 484 syntax.
• Functions and classes have useful Sphinx-style docstrings.

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
7216	6330	88%	87%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
pyquil/api/_abstract_compiler.py	92%	🟢
pyquil/api/_compiler.py	61%	🟢
pyquil/api/_qpu.py	65%	🟢
TOTAL	72%	🟢

updated for commit: 82c70ba by action🐍

[github-actions]

Bencher

Report	Tue, July 2, 2024 at 22:02:36 UTC
Project	pyquil
Branch	1786-remove-rewrite-arithmetic
Testbed	ci-runner-linux

Click to view all benchmark results

Benchmark	Latency	Latency Results nanoseconds (ns) | (Δ%)	Latency Upper Boundary nanoseconds (ns) | (%)
test/benchmarks/test_program.py::test_copy_everything_except_instructions	✅ (view plot)	10,219,576,522.80 (+0.83%)	10,322,504,986.29 (99.00%)
test/benchmarks/test_program.py::test_instructions	✅ (view plot)	3,949,730,179.60 (-0.56%)	4,081,677,608.12 (96.77%)
test/benchmarks/test_program.py::test_iteration	✅ (view plot)	3,991,505,513.00 (-0.24%)	4,157,341,626.24 (96.01%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

[MarquessV]

Deprecated the recalculation_table field to avoid a breaking change.