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

Parley Model Refactor #11

Merged
merged 3 commits into from
Oct 5, 2024
Merged

Parley Model Refactor #11

merged 3 commits into from
Oct 5, 2024

Conversation

bbengfort
Copy link
Contributor

@bbengfort bbengfort commented Oct 5, 2024

Scope of changes

Refactors the parley models to represent the latest evaluation criteria and adds user evaluations.

Type of change

  • bug fix
  • new feature
  • documentation
  • other (describe)

Acceptance criteria

This PR will be merged without review.

Author checklist

  • I have manually tested the change and/or added automation in the form of unit tests or integration tests
  • I have updated the dependencies list
  • I have added new test fixtures as needed to support added tests
  • I have added or updated the documentation

@bbengfort bbengfort changed the title Evaluation Views Parley Model Refactor Oct 5, 2024
@bbengfort bbengfort merged commit c49b5bf into main Oct 5, 2024
6 checks passed
@bbengfort bbengfort deleted the sc-27682 branch October 5, 2024 22:22
Copy link

sentry-io bot commented Oct 7, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ ImproperlyConfigured: The included URLconf '2' does not appear to have any patterns in it. If you see the 'urlpatterns'... / View Issue
  • ‼️ SyntaxError: invalid syntax (user.py, line 77) parley.models in <module> View Issue
  • ‼️ AttributeError: 'Evaluation' object has no attribute 'evaluation' / View Issue
  • ‼️ AttributeError: 'Evaluation' object has no attribute 'evaluation' /reviews/{pk} View Issue
  • ‼️ IndentationError: expected an indented block after 'if' statement on line 464 (llm.py, line 466) parley.models in <module> View Issue

Did you find this useful? React with a 👍 or 👎

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