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.

Sign up for GitHub

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

Jump to bottom

Master #1177

Merged
merged 4 commits into from
Oct 1, 2024
Merged

Master #1177

merged 4 commits into from
Oct 1, 2024

Conversation

DhruvKadam-git
Copy link
Contributor

  1. Error Handling: Added try-except blocks for file operations and network requests.

  2. Logging Enhancements: Improved logging to capture more detailed information.

  3. Code Refactoring: Created download_file and upload_index functions to avoid code repetition.

  4. Configuration: Used constants for MIN_TOKENS, MAX_TOKENS, and RECURSION_DEPTH.

DhruvKadam-git and others added 4 commits September 29, 2024 14:49
@DhruvKadam-git
Update worker.py
6698a00 
1. Error Handling: Added try-except blocks for file operations and 
    network requests.

2. Logging Enhancements: Improved logging to capture more 
    detailed information.

3. Code Refactoring: Created download_file and upload_index 
   functions to avoid code repetition.

4. Configuration: Used constants for MIN_TOKENS, MAX_TOKENS, 
    and RECURSION_DEPTH.
@DhruvKadam-git
Add .env-template
5e60495
@DhruvKadam-git
Merge remote-tracking branch 'upstream/main'
058e027 
This merge includes several important updates:

1. Error Handling: Added try-except blocks for file operations and network requests.
2. Logging Enhancements: Improved logging to capture more detailed information.
3. Code Refactoring: Created download_file and upload_index functions to avoid code repetition.
4. Configuration: Used constants for MIN_TOKENS, MAX_TOKENS, and RECURSION_DEPTH.
@DhruvKadam-git
Merge remote-tracking branch 'upstream/main'
deca772 
This merge includes several important updates:

1. Error Handling: Added try-except blocks for file operations and network requests.
2. Logging Enhancements: Improved logging to capture more detailed information.
3. Code Refactoring: Created download_file and upload_index functions to avoid code repetition.
4. Configuration: Used constants for MIN_TOKENS, MAX_TOKENS, and RECURSION_DEPTH.
@vercel Vercel
Copy link

vercel bot commented Oct 1, 2024

@DhruvKadam-git is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the application Application label Oct 1, 2024
@DhruvKadam-git
Copy link
Contributor Author

please review these and let me know if it is ok or not

@vercel Vercel
Copy link

vercel bot commented Oct 1, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 1, 2024 1:13pm

ManishMadan2882
ManishMadan2882 approved these changes Oct 1, 2024
View reviewed changes
Copy link
Collaborator

@ManishMadan2882 ManishMadan2882 left a comment

Choose a reason for hiding this comment

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

Perfect @DhruvKadam-git , Thankyou!

@dartpain dartpain merged commit 3b4a95e into arc53:main Oct 1, 2024
6 checks passed
@dartpain
Copy link
Contributor

@holopin-bot @DhruvKadam-git Thank you!

@holopin-bot Holopin Bot
Copy link

holopin-bot bot commented Oct 21, 2024

Congratulations @DhruvKadam-git, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cm2jegydo19020cl6tpkxyk6z

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

@dartpain
Copy link
Contributor

Hey @DhruvKadam-git please fill in this form if you are interested in a t-shirt
https://airtable.com/appikMaJwdHhC1SDP/pagoblCJ9W29wf6Hf/form

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ManishMadan2882 ManishMadan2882 ManishMadan2882 approved these changes

Assignees
No one assigned
Labels
application Application
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DhruvKadam-git @dartpain @ManishMadan2882