Skip to content

Move Fermi-Hubbard publication data fetching - #488

Open
Mounika-2604 wants to merge 1 commit into
quantumlib:mainfrom
Mounika-2604:fix/issue-125
Open

Mounika-2604 wants to merge 1 commit into
quantumlib:mainfrom
Mounika-2604:fix/issue-125

Conversation

@Mounika-2604

@Mounika-2604 Mounika-2604 commented Oct 5, 2026 •

Copy link
Copy Markdown

Resolves #125.

Summary

Following the decision in #125 to keep publication data fetching in the experiment modules:

  • Added get_publication_data as an alias for fetch_publication_data in recirq/fermi_hubbard/publication.py.
  • Re-exported both functions in recirq/fermi_hubbard/__init__.py.
  • Added a unit test in publication_test.py to verify the alias and export.

Testing

  • Ran pytest locally on recirq/fermi_hubbard/ (all tests passed).

@Mounika-2604

Copy link
Copy Markdown
Author

Hi @mhucka, the changes for this issue are ready. Could you please review the PR when you have a chance? Thank you!

@mhucka mhucka left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description is longer than the changes in the PR itself. It was obviously written by an AI agent. Don't just copy-paste the output from an agent. Summarize what is being done in your own words. Please respect the time of your human reviewers.

@Mounika-2604

Copy link
Copy Markdown
Author

I apologize for that, @mhucka. You're completely right — the previous description was unnecessarily verbose for such a small change.

I have updated the PR description with a concise summary of what was actually changed. Thank you for pointing that out and for taking the time to review.

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.

Decide where example-data-fetching should live

2 participants