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

Hotfix/v2024.11.23 #924

Merged
merged 45 commits into from
Nov 25, 2024
Merged

Hotfix/v2024.11.23 #924

merged 45 commits into from
Nov 25, 2024

Conversation

surapuramakhil
Copy link
Contributor

No description provided.

waterstark and others added 30 commits November 5, 2024 16:03
udpate v3.1.0 with Release/v11.15.2024
repalced union with base class.
Some providers (as OpenAI) support reducing cost of calls by caching repetitive content, but the feature works only with the initial portion (prefix) of prompts
included links to AI/ML API docs and the endpoint to be used in the code.
perf: ⚡ Optimize prompt for better caching
(cherry picked from commit 6b23715)
(cherry picked from commit c75f78f)
[FEATURE] Log time in success, failed, skipped and data json files

Co-authored-by: NamamiShanker <[email protected]>
Sync Release branch with main
LinkedIn has changed their HTML structure therefore app stopped to apply to jobs

new
Test had been failing after removing unnecessary wheel ( .find_elements()[0] )

n
Copy link
Collaborator

@49Simon 49Simon left a comment

Choose a reason for hiding this comment

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

Tested it and its applying to jobs. Once these issues are addressed it can be merged as patch.

src/ai_hawk/job_manager.py Outdated Show resolved Hide resolved
src/ai_hawk/job_manager.py Show resolved Hide resolved
@surapuramakhil
Copy link
Contributor Author

surapuramakhil commented Nov 24, 2024

@49Simon reviews typically happen when code goes to release / hotfix branches. This is mostly for testing purposes, more like release branch should be working on at least two others.

Appreciate your input though

@49Simon
Copy link
Collaborator

49Simon commented Nov 24, 2024

You are merging new code into the main branch as a critical patch, skipping the usual release process. It’s still important to fully test and review the code.

@surapuramakhil surapuramakhil dismissed 49Simon’s stale review November 25, 2024 13:26

Addressed review comments

@surapuramakhil surapuramakhil merged commit 892a6fb into main Nov 25, 2024
2 checks passed
@surapuramakhil surapuramakhil deleted the hotfix/v2024.11.23 branch November 25, 2024 13:35
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.

9 participants