Removing local resources estimator

microsoft / Quantum

Microsoft Quantum Development Kit Samples

https://docs.microsoft.com/quantum

MIT License

3.86k stars 918 forks source link

Removing local resources estimator #791

Closed DmitryVasilevsky closed 1 year ago

DmitryVasilevsky commented 1 year ago

Removed use of deprecated ResourcesEstimator class.

DmitryVasilevsky commented 1 year ago

This seems good enough to compile, but I'm not sure if it changes the behavior of the samples too much. Perhaps we should consider removing these two samples entirely.

For the Estimate function change, QCTraceSimultor still produces the results, but aggregated results are missing (because Console.WriteLine(estimator.ToTSV()); has to be removed). Since we are keeping QCTraceSimulator I think it makes sense to keep these samples.