Skip to content

[Pasqal] Simple job to batch mapping#60

Draft
awennersteen wants to merge 6 commits intomainfrom
aw/job-to-batch
Draft

[Pasqal] Simple job to batch mapping#60
awennersteen wants to merge 6 commits intomainfrom
aw/job-to-batch

Conversation

@awennersteen
Copy link
Collaborator

Description of Change

Checklist ✅

  • Have you included a description of this change?
  • Have you updated the relevant documentation to reflect this change?
  • Have you made sure CI is passing before requesting a review?

Ticket

  • Fixes #
  • Is Part of #

@awennersteen awennersteen self-assigned this Feb 25, 2026
Base automatically changed from aw/fix-task-status to main February 25, 2026 13:50
.job_to_batch
.get(task_id)
.ok_or_else(|| anyhow!("No batch mapping found for Pasqal job id '{}'", task_id))?;
match self.api_client.cancel_batch(batch_id).await {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can cancel the job instead of canceling the batch which will have the same effect. You won't need to store the batch_id in that case

.job_to_batch
.get(task_id)
.ok_or_else(|| anyhow!("No Batch mapping found for Pasqal job id '{}'", task_id))?;
match self.api_client.get_batch_results(batch_id).await {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unfortunately there is no endpoint in the cloud to get job results directly. You can get a s3 link using this route but for air-gapped nodes we prefer downloading the results server side.

I guess this can be refined later once we look into creating a batch in the acquire method as discussed in this #59

@awennersteen awennersteen marked this pull request as draft February 25, 2026 15:30
@awennersteen awennersteen marked this pull request as draft February 25, 2026 15:30
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.

2 participants