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

[BUG] Still getting this error when trying to run deployed flow: Execution failure: (TypeError) unsupported operand type(s) for +: 'int' and 'NoneType'" #3806

Closed
Chriss-24 opened this issue Oct 9, 2024 · 3 comments · May be fixed by #3825
Closed

[BUG] Still getting this error when trying to run deployed flow: Execution failure: (TypeError) unsupported operand type(s) for +: 'int' and 'NoneType'" #3806

Chriss-24 opened this issue Oct 9, 2024 · 3 comments · May be fixed by #3825
Labels
bug Something isn't working no-recent-activity There has been no recent activity on this issue/pull request

Comments

@Chriss-24
Copy link

Chriss-24 commented Oct 9, 2024 •
edited
Loading

Everything related to this issue was marked as resolved but it is resolved and I am still getting this error when I try to consume prompt flow via http request.

When I am adding openai<=1.44.1 to the requirements.txt (as it was suggested in other topics) I can't deploy a flow and getting this error: Authorization failed for container registry to access image details. Make sure that the container registry associated with the Environment is not unintentionally private, or if this container registry is behind a private network, ensure that it has been set up properly. Otherwise, please try running resync keys on your workspace and retry
@Chriss-24 Chriss-24 added the bug Something isn't working label Oct 9, 2024
@hiroki0525
Copy link
Contributor

I'm still encountering a similar bug, the version of promptflow is 1.16.1.

promptflow._core._errors.ToolExecutionError: Execution failure in 'check_japanese_foods_from_table': (TypeError) unsupported operand type(s) for +: 'dict' and 'dict'
2024-10-10 22:44:51 +0900   42295 execution.flow     WARNING  Failed to calculate metrics due to exception: unsupported operand type(s) for +: 'int' and 'dict'.
2024-10-10 22:44:51 +0900   42295 execution.flow     WARNING  Failed to calculate metrics due to exception: unsupported operand type(s) for +: 'int' and 'dict'.
2024-10-10 22:44:51 +0900   42295 execution          INFO     Node create_version in line 0 is canceled.
pf.flow.test failed with UserErrorException: TypeError: Execution failure in 'check_japanese_foods_from_table': (TypeError) unsupported operand type(s) for +: 'dict' and 'dict'

@darrelhong
Copy link

darrelhong commented Oct 11, 2024 •
edited
Loading

it looks like the tokens object can have nested objects as well, like

{'completion_tokens': 298, 'prompt_tokens': 3193, 'total_tokens': 3491, 'completion_tokens_details': {'audio_tokens': None, 'reasoning_tokens': 0}

A solution would be to have a helper function that merges nested objects as well
Let me know if I could pick this up

@WilliamWatsonEAI WilliamWatsonEAI mentioned this issue Oct 18, 2024
Open
@darrelhong darrelhong mentioned this issue Oct 19, 2024
Open
8 tasks
@scherici scherici mentioned this issue Oct 23, 2024
Closed
@github-actions GitHub Actions
Copy link

Hi, we're sending this friendly reminder because we haven't heard back from you in 30 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 7 days of this comment, the issue will be automatically closed. Thank you!

@github-actions github-actions bot added the no-recent-activity There has been no recent activity on this issue/pull request label Nov 10, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working no-recent-activity There has been no recent activity on this issue/pull request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: improve merge token logic to account for nested dicts darrelhong/promptflow
3 participants
@darrelhong @hiroki0525 @Chriss-24