Skip to content

Refactor codebase for Trip Advisor Dataset Loader - #48

Merged
jkawamoto merged 3 commits into
mainfrom
tripadvisor
Mar 21, 2025
Merged

Refactor codebase for Trip Advisor Dataset Loader#48
jkawamoto merged 3 commits into
mainfrom
tripadvisor

Conversation

@jkawamoto

Copy link
Copy Markdown
Member

Summary

This pull request consolidates several refactoring changes to improve code quality, readability, and usability across the project:

  • PEP 8 Compliance: Reformatted code to enforce a maximum line length of 79 characters for better readability and adherence to Python's style guide (pyproject.toml updated accordingly).
  • Codebase Cleanup: Removed outdated and unused scripts run_greedy.py and testMinTree.py to simplify the project structure.
  • Improved Method Flexibility: Enhanced the new_reviewer and add_review methods by adding optional parameters (anomalous_score and _time, respectively) to accommodate more flexible use cases.

Additional Notes

These changes streamline the codebase and enhance its maintainability while preserving current functionality.

Updated `new_reviewer` to accept an optional `anomalous_score` and default it to 0, allowing more flexibility during reviewer creation. Modified `add_review` to include an optional `_time` parameter for more precise review tracking. These changes enhance method usability and support for additional data.
These files were removed as they are either outdated, redundant, or no longer necessary for the current functionality of the project. Their removal helps maintain a cleaner and more manageable codebase.
This update modifies various files to reformat code for a maximum line length of 79 characters, as specified in the updated `pyproject.toml`. The changes improve readability and adhere to Python's PEP 8 style guide.
@jkawamoto
jkawamoto merged commit 96a49c4 into main Mar 21, 2025
@jkawamoto
jkawamoto deleted the tripadvisor branch March 21, 2025 04:44
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