Project refactor #161

jfrverdasca · 2024-12-05T11:47:22Z

What this PR change:

Refactor of root llm.py file
Refactor of root repo.py file
Refactor of root run.py file
Refactor of tasks
litellm_service package refactor
Redis now use variables to avoid key names errors
Little bug fix

Reviewers:

Issues:

Schemas refactor and files check #152

Print screens showing the changes:

NA

…ode don't add any value and just creates entropy * Renamed github_token variable inside GithubRequests and github.py file to just token

…straction

.env.sample

docker-compose.yml

labs/parsers/response_parser.py

labs/tasks/redis_client.py

labs/tasks/checks.py

labs/tasks/redis_client.py

labs/tests/test_llm.py

labs/tasks/redis_client.py

labs/tasks/repository.py

labs/tasks/run.py

labs/parsers/response_parser.py

jfrverdasca added 5 commits December 4, 2024 11:53

Fix: Refactor to remove repo.py and run.py from project root. Their c…

12ae64b

…ode don't add any value and just creates entropy * Renamed github_token variable inside GithubRequests and github.py file to just token

Fix: Replace the usage of vanilla redis client to new redis client ab…

d774be0

…straction

Fix: Replace the usage of vanilla redis client to new redis client ab…

a648a81

…straction

Fix: Refactor the llm.py file in the project root

9e2bafb

Fix: Package name refactor

173d378

jfrverdasca requested review from kenvontucky and JdFSilva December 5, 2024 11:47

jfrverdasca self-assigned this Dec 5, 2024

jfrverdasca added 3 commits December 5, 2024 15:51

Fix: Test fix

cb54540

Fix: Ruff fix

d0a4d42

Fix: Ruff fix

9cc661a

JdFSilva reviewed Dec 6, 2024

View reviewed changes

kenvontucky requested changes Dec 9, 2024

View reviewed changes

labs/tasks/run.py Outdated Show resolved Hide resolved

labs/parsers/response_parser.py Outdated Show resolved Hide resolved

jfrverdasca added 2 commits December 9, 2024 16:39

Fix: Pull request comments (mainly variable renaming)

fca0fd4

Fix: String typo

2c4b5a9

JdFSilva approved these changes Dec 10, 2024

View reviewed changes

jfrverdasca added 4 commits December 10, 2024 11:08

Fix: Jupyter notebook repo variables

28e59e4

Fix: Little variable fix

9ea3248

Fix: Typing fix

b4f4317

Fix: Typing fix

8ec9d46

kenvontucky approved these changes Dec 10, 2024

View reviewed changes

jfrverdasca merged commit a4e4948 into main Dec 10, 2024
1 check passed

jfrverdasca deleted the jv/152/repo_and_run_files_refactor branch December 10, 2024 12:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Project refactor #161

Project refactor #161

jfrverdasca commented Dec 5, 2024

Project refactor #161

Project refactor #161

Conversation

jfrverdasca commented Dec 5, 2024

What this PR change:

Reviewers:

Issues:

Print screens showing the changes: