Skip to content

Implement getting reason a record is waiting#759

Merged
bennybp merged 1 commit into
mainfrom
waiting_reason
Sep 29, 2023
Merged

Implement getting reason a record is waiting#759
bennybp merged 1 commit into
mainfrom
waiting_reason

Conversation

@bennybp
Copy link
Copy Markdown
Contributor

@bennybp bennybp commented Sep 24, 2023

Description

Implement functionality for testing why a record is in the waiting status. Inspired by openmm/spice-dataset#82 (comment)

Often, a record will be waiting because managers don't exist or have the wrong tags/programs. This was opaque to the user. This allows for the user to test why a record is waiting.

r = client.get_record(record_id)
reason = r.get_waiting_reason()
print(reason)

# or

reason = client.get_waiting_reason(record_id)
print(reason)

Changelog description

Implement functionality for checking why a record is in the waiting state

Status

  • Code base linted
  • Ready to go

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 24, 2023

Codecov Report

Merging #759 (a4a9a31) into main (e8d9cba) will decrease coverage by 0.04%.
The diff coverage is 75.75%.

Additional details and impacted files

@bennybp bennybp changed the title [HOLD] Implement getting reason a record is waiting Implement getting reason a record is waiting Sep 29, 2023
@bennybp bennybp merged commit 23c4103 into main Sep 29, 2023
@bennybp bennybp deleted the waiting_reason branch September 29, 2023 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant