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

[Update] Refactor trace visualization components: introduce pipeline structure, replace evaluators with steps, and enhance depth management #7

Merged
merged 1 commit into from
Dec 29, 2024

Conversation

patrick204nqh
Copy link
Owner

@patrick204nqh patrick204nqh commented Dec 29, 2024

Description

  • Refactor trace visualization components by introducing a pipeline structure, replacing evaluators with steps, and enhancing depth management.
  • Update README.md to include an open attribute in the <details> tag.
  • Replace evaluators and the depth manager with steps in the BaseCollector class.
  • Delete obsolete DepthManager and evaluator classes.
  • Introduce new steps for trace data processing: AssignDepthStep, ValidationStep, HiddenStep, and LinkingStep.
  • Enhance TracePipeline and TracePipelineBuilder to manage trace data processing steps.
  • Refactor method parameter extraction and logging in method_call.rb.

Type of Change

Check all that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing behavior)
  • Documentation update
  • Other (please describe):

How Has This Been Tested?

Describe the tests you ran to verify your changes:

  • Running bundle exec rspec passes all tests
  • Added or updated specs for new functionality
  • Tested changes on different Ruby versions if relevant

If you have any manual testing steps, please detail them here.

Checklist

  • I have followed the Ruby Style Guide.
  • I have performed a self-review of my code.
  • I have added or updated tests as needed.
  • All existing and new tests pass.
  • I have updated the documentation (if needed), including README.md and CHANGELOG.md.
  • I have considered the impact on users and documented any required changes or upgrades.

Deployment Notes

No

Related Issues

No

Screenshots (if applicable)

No

…, replace evaluators with steps, and enhance depth management
@patrick204nqh patrick204nqh added documentation Updates or changes related to documentation. enhancement A new feature or an improvement to existing functionality. labels Dec 29, 2024
@patrick204nqh patrick204nqh self-assigned this Dec 29, 2024
@patrick204nqh patrick204nqh added in review Pull requests currently under review. approved PRs that are approved and ready to merge. and removed in review Pull requests currently under review. labels Dec 29, 2024
@patrick204nqh patrick204nqh merged commit a159a0a into main Dec 29, 2024
6 checks passed
@patrick204nqh patrick204nqh deleted the develop branch December 29, 2024 10:03
@patrick204nqh patrick204nqh added this to the Version 1.0.0 Release milestone Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved PRs that are approved and ready to merge. documentation Updates or changes related to documentation. enhancement A new feature or an improvement to existing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant