Skip to content

ingestion of qscout results objects#1367

Merged
richrines1 merged 11 commits into
mainfrom
rich/qscout-results
May 5, 2026
Merged

ingestion of qscout results objects#1367
richrines1 merged 11 commits into
mainfrom
rich/qscout-results

Conversation

@richrines1

Copy link
Copy Markdown
Contributor

No description provided.

@richrines1 richrines1 marked this pull request as ready for review April 16, 2026 16:39
@@ -1 +1,3 @@
checks-superstaq~=0.5.64
jaqalpaq>=1.3.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do these need to be in the css and qss requirements if they get pulled in anyway from gss?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

currently the dev options don't pull from one another (so e.g. cirq-superstaq[dev] still just requires general-superstaq and not general-superstaq[dev]). i think the idea is that each <package>[dev] should only add the dependencies necessary to run checks on , and not on other *-superstaq packages.. (but not sure how much of a difference this makes in practice atm)

Comment thread cirq-superstaq/cirq_superstaq/job.py Outdated
return single_counts

def _terminal_measurement_qubit_indices(self, index: int) -> list[int]:
"""Returns the ordered physical qubit indices for each measurement in a compiled circuit.

@cdbf1 cdbf1 Apr 17, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: args and return docs (although its a private function so I guess somewhat optional)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

@bharat-thotakura bharat-thotakura left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread general-superstaq/general_superstaq/job.py Outdated
richrines1 and others added 2 commits May 4, 2026 15:34
Co-authored-by: Bharath Thotakura <113555655+bharat-thotakura@users.noreply.github.com>
@richrines1 richrines1 merged commit 36fef65 into main May 5, 2026
21 checks passed
@richrines1 richrines1 deleted the rich/qscout-results branch May 5, 2026 15:09
richrines1 added a commit that referenced this pull request May 6, 2026
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.

3 participants