Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Infographic for the main quantum open source projects #106

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jyotiraj-code
Copy link

Solves issue #93. In the PR:

@victor-onofre
Copy link
Contributor

Hi @jyotiraj-code! This looks great. I have a couple of comments that would improve the work:

  • Delete the Quantum Compilers and Assembly section. Right now, it is complicated to separate these terms from the full frameworks
  • Mitiq is not a compiler, it is a framework for error mitigation. You can add a section of Quantum Noise and add Mitiq and other libraries related to error mitigation and correction
  • In Quantum Algorithms, you can delete the Qiskit packages. There is too much Qiskit in the image. Being in the full-stack and simulators is fine. Here, you can check the projects of the unitary hack to add more frameworks. Examples are OpenQAOA, sQUlearn, Qadence, and Qrisp.
  • In Quantum Simulators, you can add Pulser, Bloqade, Perceval, etc. (Check the unitary hack projects)
  • The post quantum cryptography is a great idea
  • What do you mean by experimental quantum computing?

Thanks for your help!

Let me know if you have any questions.

@s-peryt
Copy link
Contributor

s-peryt commented Jun 8, 2024

Good job @jyotiraj-code ! Few comments:

  1. Please review typos and formatting. Especially make sure that each box starts with capital letter.
  2. I'd recommend putting all of the boxes in alphabetical order in each section. I see you mostly tried to do that, but it's not everywhere.
  3. I don't get the usage of labels. In "Quantum full-stack libraries" they might have some sense, although I am not fully convinced that all of them have, but in case of "Quantum Simulators" why duplicate the same info. I.e. "Simulator" in section header and labels? Same for all remaining sections.
  4. In "Quantum and post quantum cryptography" there is duplicate of "OpenSSH"
  5. There is something wrong IMHO with "Arline Benchmarks" in "Quantum Assembly" section. My best guess is that it should be something else.
  6. I am not sure if the whole "Quantum Algorithms" section should be called like this. First of all I'll get rid of Qiskit ones as those can be multiple as mentioned by @victor-onofre . Second, it's missing simple way to Google "Adapt". If you'll decide to leave it maybe put is as "BBN-Q/Adapt". Then you should probably do the same with QuantumTomography.jl. But again IMHO I think this whole section should be modified as the algorithms in general are the most broad area probably on this whole list.

That's probably all for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants