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

Refactor/134 code check Trace Testing Environment #141

Merged
merged 13 commits into from
May 25, 2024

Conversation

FR-SON
Copy link
Collaborator

@FR-SON FR-SON commented May 23, 2024

This PR implements refactoring for the trace_comparator app. The following changes were made:

  • update docstrings, to be more descriptive in case of complex functions
  • split large functions into multiple smaller ones
  • add typehints where necessary
  • remove code duplication by extracting and reusing functionality
  • rename variables to be more concise
  • fix indents in html files for better readability

This closes #134

@FR-SON FR-SON self-assigned this May 23, 2024
@tkv29
Copy link
Collaborator

tkv29 commented May 23, 2024

could you please drop the metrics in the view after the extraction for the trace testing env and in the trace testing env result view

image image image

@tkv29 tkv29 requested review from nils-schmitt, tkv29 and thangixd May 23, 2024 17:55
@thangixd
Copy link
Collaborator

the code looks fine to me gj!

@thangixd thangixd self-requested a review May 24, 2024 11:03
FR-SON added 3 commits May 24, 2024 13:54
# Conflicts:
#	tracex_project/db.sqlite3
#	tracex_project/extraction/fixtures/prompts_fixture.json
#	tracex_project/extraction/logic/orchestrator.py
Copy link
Collaborator

@nils-schmitt nils-schmitt left a comment

Choose a reason for hiding this comment

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

Very good work, just minor changes requested

tracex_project/trace_comparator/comparator.py Outdated Show resolved Hide resolved
tracex_project/trace_comparator/comparator.py Outdated Show resolved Hide resolved
tracex_project/trace_comparator/comparator.py Outdated Show resolved Hide resolved
tracex_project/trace_comparator/comparator.py Outdated Show resolved Hide resolved
tracex_project/trace_comparator/comparator.py Outdated Show resolved Hide resolved
tracex_project/trace_comparator/views.py Show resolved Hide resolved
Copy link
Collaborator

@nils-schmitt nils-schmitt left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@FR-SON FR-SON merged commit 0dee86c into main May 25, 2024
2 checks passed
@nils-schmitt nils-schmitt deleted the refactor/134-code-check-tte branch May 25, 2024 16:49
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.

Code check: Trace Testing Environment
4 participants