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

Jump to bottom

Add option for synchronous evaluation #6

Merged
merged 8 commits into from
Jun 26, 2024
Merged

Add option for synchronous evaluation #6

merged 8 commits into from
Jun 26, 2024

Conversation

soumik12345
Copy link
Collaborator

@soumik12345 soumik12345 commented Jun 22, 2024 •
edited
Loading

This PR makes the following changes:

  • Split up the legacy HemmEvaluation class into 2 classes: AsyncHemmEvaluation and HemmEvaluation for async and sequential execution respectively.
  • Add async method evaluate_async to the metrics.
  • Add async method infer_async to EvaluationPipeline.
  • Add option to disable safety checker in BaseDiffusionModel.
  • Update readme to cite more sources of inspiration.

@soumik12345 soumik12345 self-assigned this Jun 22, 2024
@soumik12345 soumik12345 merged commit ac4daae into main Jun 26, 2024
2 checks passed
@soumik12345 soumik12345 deleted the feat/sync-eval branch June 26, 2024 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ayulockin ayulockin Awaiting requested review from ayulockin

Assignees

@soumik12345 soumik12345

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@soumik12345