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.

Sign up for GitHub

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

fix fermi_hubbard/publication_test by updating datadryad download URLs since they appear to have changed #166

Merged
merged 10 commits into from
Apr 28, 2021
Merged

fix fermi_hubbard/publication_test by updating datadryad download URLs since they appear to have changed #166

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
a95717e
add a library containing mcpe calculation utilities, DependencyLists …
weinstein Feb 12, 2021
6c10bae
Merge remote-tracking branch 'upstream/master'
weinstein Feb 16, 2021
6190fad
remove iswap factory and replace swap factory with sqrt-iswap decompo…
weinstein Feb 16, 2021
396e946
fix typo
weinstein Feb 16, 2021
b350914
replace _adjacent_qubits() with GridQubit.neighbors() and remove Swap…
weinstein Feb 17, 2021
f1a3882
replace assert's with ValueError's, update manhattan distance TODO, a…
weinstein Feb 19, 2021
dc50145
refactor swap updater tests
weinstein Feb 19, 2021
41aa49e
Merge remote-tracking branch 'upstream/master'
weinstein Mar 2, 2021
a5da396
Merge remote-tracking branch 'upstream/master'
weinstein Apr 1, 2021
770968d
update datadryad download URLs since they appear to have changed
weinstein Apr 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions recirq/fermi_hubbard/publication.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -241,10 +241,10 @@ def fetch_publication_data(

base_url = "https://datadryad.org/stash/downloads/file_stream/"
data = {
"gaussians_1u1d_nofloquet": "706210",
"gaussians_1u1d": "706211",
"trapping_2u2d": "706212",
"trapping_3u3d": "706213"
"gaussians_1u1d_nofloquet": "451326",
"gaussians_1u1d": "451327",
"trapping_2u2d": "451328",
"trapping_3u3d": "451329"
Comment on lines +244 to +247
Copy link
Contributor

Choose a reason for hiding this comment

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

It's a bummer these aren't permanent identifiers. I reached out to dryad help and they essentially said the only thing permanent is the dataset DOI. So a long-term fix will probably need to locate files from the DOI instead of from /stash/downloads/file_stream.

}
if exclude is not None:
data = {path: key for path, key in data.items() if path not in exclude}
Expand Down