search

quil-lang / qvm

The high-performance and featureful Quil simulator.
Other
411 stars 57 forks source link

remove qvm-ng #300

Closed stylewarning closed 1 year ago

stylewarning commented 1 year ago

Sorry to see it go, but it has not been developed, and it is not used in production.

Fixes #299 #296

ecpeterson commented 1 year ago

I'm also sorry to see it go, but it is what it is.

Do the tests still pass locally? The runner is still mad about some package renaming in CL-QUIL.

stylewarning commented 1 year ago

No they don't. Some JSON library API-induced bit rot. It's probably fixable tho without too much work.

If you or someone else uses it, I can fix the tests and keep it going.

ecpeterson commented 1 year ago

Oh I meant the tests on the rest of QVM, after the NG deletion is applied — the thing the GitHub test runner is trying & failing to do.

stylewarning commented 1 year ago

Oh yes, they should be, but I'll have to double check.

We need to change test runners desperately so that they're meaningful. :S

notmgsk commented 1 year ago

👋🏽 😿

notmgsk commented 1 year ago

never forget QVMaaS.

appleby commented 1 year ago

pqvm ftw

stylewarning commented 1 year ago

pqvm ftw

d(-_-)bvm

appleby commented 1 year ago

pqvm ftw

d(-_-)bvm

p(-_-)qvm

persistent-but-only-in-our-hearts-qvm

Maybe the real treasure was the PERSISTENT-QVMs we made along the way...