feat: Add task processing API #12

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

Merged

marcelklehr merged 1 commit into main from feat/task-processing

Jul 17, 2024

Member

provokateurin commented May 17, 2024

Still very much WIP.

provokateurin force-pushed the feat/task-processing branch 2 times, most recently from d4e7a60 to e285cf0 Compare

June 4, 2024 11:00

provokateurin requested review from andrey18106 and bigcat88

June 4, 2024 11:16

provokateurin commented

View reviewed changes

lib/main.py Outdated Show resolved Hide resolved

bigcat88 reviewed

View reviewed changes

lib/main.py Outdated Show resolved Hide resolved

provokateurin commented

View reviewed changes

lib/main.py Outdated Show resolved Hide resolved

provokateurin requested a review from bigcat88

June 7, 2024 06:41

bigcat88 reviewed

View reviewed changes

lib/main.py Outdated Show resolved Hide resolved

Member

marcelklehr commented Jun 8, 2024

Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/home/marcel/code/llm2/lib/main.py", line 46, in run
    response = nc.providers.task_processing.next_task(list(provider_ids), list(task_type_ids))
AttributeError: 'ProvidersApi' object has no attribute 'task_processing'. Did you mean: 'text_processing'?

Is this not available in v0.13 yet?

Member

bigcat88 commented Jun 8, 2024

Is this not available in v0.13 yet?

it is availably in PR: cloud-py-api/nc_py_api#254

AppAPI PR was not merged yeat

Member Author

provokateurin commented Jun 17, 2024

To test this please locally revert a16e617

provokateurin force-pushed the feat/task-processing branch from 24570f8 to d20d634 Compare

July 8, 2024 08:57

provokateurin marked this pull request as ready for review

July 8, 2024 08:57

Member Author

provokateurin commented Jul 8, 2024

Still depends on cloud-py-api/nc_py_api#254, but I could finally test it correctly!

provokateurin requested review from bigcat88 and marcelklehr

July 8, 2024 08:57

Member

bigcat88 commented Jul 9, 2024

Still depends on cloud-py-api/nc_py_api#254, but I could finally test it correctly!

nc-py-api 0.14.0 with the support of the new API was published

we can test this PR with it and free to merge this after that.

bigcat88 reviewed

View reviewed changes

lib/main.py Outdated Show resolved Hide resolved

bigcat88 reviewed

View reviewed changes

lib/main.py Show resolved Hide resolved

bigcat88 reviewed

View reviewed changes

lib/main.py Show resolved Hide resolved

provokateurin force-pushed the feat/task-processing branch from d20d634 to e5fe1f6 Compare

July 11, 2024 07:55


          feat: Add task processing API

f29f439

Signed-off-by: provokateurin <[email protected]>

provokateurin force-pushed the feat/task-processing branch from e5fe1f6 to f29f439 Compare

July 11, 2024 08:49

provokateurin requested a review from bigcat88

July 11, 2024 08:54

marcelklehr approved these changes

View reviewed changes

Member

marcelklehr left a comment

Looks, good. Thanks!

marcelklehr merged commit 1add161 into main

marcelklehr deleted the feat/task-processing branch

July 17, 2024 09:57

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

Labels

None yet