-
Notifications
You must be signed in to change notification settings - Fork 425
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
Allow chunking map call to take in sublists of length 10 and support lists as inputs #939
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The command that failed is
This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The command that failed is
This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The command that failed is
This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. I'm getting the same errors 3 times in a row, so I will stop working on fixing this PR. |
1 similar comment
GitHub actions yielded the following error. The command that failed is
This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. I'm getting the same errors 3 times in a row, so I will stop working on fixing this PR. |
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
|
GitHub actions yielded the following error. The relevant lines from the logs corresponding to the error are:
The command that failed is This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The relevant lines from the logs corresponding to the error are:
The command that failed is This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
GitHub actions yielded the following error. The relevant lines from the logs corresponding to the error are:
The command that failed is This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. I'm getting the same errors 3 times in a row, so I will stop working on fixing this PR. |
pyproject.toml
Outdated
PyGithub = "1.58.2" | ||
loguru = "^0.6.0" | ||
python = "^3.10" # or another compatible version | ||
PyGithub = "1.58.3" # or another compatible version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
revert
|
GitHub actions yielded the following error. The relevant lines from the logs are:
The command that failed is There are a lot of errors. This is likely a larger issue with the PR and not a small linting/type-checking issue. This is likely a linting or type-checking issue with the source code but if you are updating the GitHub Actions or versioning, this could be an issue with the GitHub Action yaml files. |
sweepai/utils/utils.py
Outdated
@@ -215,6 +215,24 @@ def __enter__(self): | |||
|
|||
@method() | |||
def chunk( | |||
self, | |||
file_content: Union[str, List[str]], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use | instead of Union and list instead of List
Description
This PR modifies the map function in the chunker to support sublists of length 10 and lists as inputs. The current implementation of the map function in
utils/utils.py
does not handle these types of inputs. This PR adds the necessary checks and logic to process each item in the list or sublist accordingly.Summary of Changes
Please review and merge this PR. Thank you!
Fixes #936.
To checkout this PR branch, run the following command in your terminal:
To make tweaks to this pull request, leave a comment below or in the code.