Skip to content

Automatic experiment monitor#521

Open
coruscating wants to merge 10 commits into
qiskit-community:mainfrom
coruscating:job-monitor
Open

Automatic experiment monitor#521
coruscating wants to merge 10 commits into
qiskit-community:mainfrom
coruscating:job-monitor

Conversation

@coruscating
Copy link
Copy Markdown
Collaborator

Summary

Adds monitor() to experiment data, which displays the current status from status(). Run by calling exp_data.monitor().

@coruscating coruscating added the needs review needs review label Nov 12, 2021
@coruscating
Copy link
Copy Markdown
Collaborator Author

After talking with Chris, will update this to show the position in the queue of the current job and the total number of jobs.

@coruscating coruscating removed the needs review needs review label Dec 9, 2021
@coruscating coruscating added this to the Release 0.3 milestone Dec 9, 2021
Copy link
Copy Markdown
Collaborator

@chriseclectic chriseclectic left a comment

Choose a reason for hiding this comment

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

This PR needs some refactoring after #599 which changed some of these methods significantly. Now there is separate methods for job_status and analysis_status which you could use to get the status for each part separate (and status is combined one for full experiment). For the analysis monitoring part you could use the _analysis_callbacks ordered dict to find the first AnalysisCallback.status that is currently running to get the i/N part for analysis.

I also wonder if we should try and use a package like https://github.com/tqdm/tqdm
to make a progress bar for the monitor.

The output is in format "Experiment Status:" followed by one of the possible
* JOB (1/N) `STATUS` if the experiment is in the job phase. For full possible statuses of
each job, see the :meth:`status` method.
* ANALYSIS IN PROGRESS if the experiment is in the analysis phase.
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.

After #599 this could be changed to something like Analysis (1/N) STATUS` like for jobs, since you should be able to see how many analysis jobs there are and get the status analysis status.

@chriseclectic chriseclectic removed this from the Release 0.3 milestone Apr 6, 2022
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 18, 2023

CLA assistant check
All committers have signed the CLA.

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