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.

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

A/B Testing Notebook #155

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

A/B Testing Notebook #155

wants to merge 3 commits into from

Conversation

dmarx
Copy link
Contributor

@dmarx dmarx commented Dec 11, 2022

To do:

Blocking

  • fix save_favorite_only behavior
  • Make notebook more robust to errors, resuming
    • read outcomes.log if it exists, use it to populate running_score, SAMPLE_IDX, etc.

Potential Future Features

  • richer logging
    • log time taken for generation
      • maybe writing images to gdrive is slower than writing them locally?
    • log time taken by user to choose
    • log incidence of safety mitigation trigger
    • log incidence of errors and conditions that emitted them
    • record "pass" - i.e. when user regenerates without selecting a favorite
  • optionally use different seed for samples
  • optionally request multiple samples per test case
  • use plotly (or some such) to permit user to interactively facet histogram on combinatorial options
  • Additional rating dimensions
    • aesthetic preference
    • relevance to prompt
    • least weird / most devoid of artifacts
    • best fit to style
    • best fit to content
    • best foreground
    • best background
    • best composition
    • best small details
    • best texture
    • most photorealistic
    • most artistic

@sonarcloud
Copy link

sonarcloud bot commented Dec 11, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

1 participant