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

Integrate optimizers into compiling XXZ code #53

Merged
merged 150 commits into from
Jun 26, 2024
Merged

Integrate optimizers into compiling XXZ code #53

merged 150 commits into from
Jun 26, 2024

Conversation

MatteoRobbiati
Copy link
Collaborator

This leave the possibility to use the old GD but also enables to use more optimizers.

andrea-pasquale and others added 30 commits May 29, 2024 07:02
…ergy by hand. Later something like this will be the documentation of the qibo model
@marekgluza
Copy link
Contributor

@MatteoRobbiati I think this is integrated into #52 and #55 ? I closed #34 because it will be impossible to merge that. Please close or move the target branch to those of Andrea, thanks!

@MatteoRobbiati
Copy link
Collaborator Author

@MatteoRobbiati I think this is integrated into #52 and #55 ? I closed #34 because it will be impossible to merge that. Please close or move the target branch to those of Andrea, thanks!

Not yet but we are going to integrate it soon, thus this one can be closed. @andrea-pasquale can you confirm my modifications done in utils.py have been migrated into your branches?

@andrea-pasquale
Copy link
Collaborator

Yes, to be sure you can rebase this PR to main and I will merge everything here.

@andrea-pasquale
Copy link
Collaborator

You can try to run stuff in #55 using --optimization_method to double check.

@andrea-pasquale andrea-pasquale changed the base branch from compiling_XXZ to main June 26, 2024 09:34
@andrea-pasquale andrea-pasquale merged commit 7803314 into main Jun 26, 2024
@Edoardo-Pedicillo Edoardo-Pedicillo deleted the gci_opt branch October 15, 2024 08:59
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.

8 participants