aljaz/dev-gpt
1
0
Fork 0
mirror of https://github.com/aljazceru/dev-gpt.git synced 2025-12-25 01:14:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: executor name dockerfile files previous error

Browse Source
This commit is contained in:
Joschka Braun
2023-04-14 14:18:00 +02:00
parent 3ea75553e1
commit 9b9aa953d2
2 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/executor_factory.py
View File

@@ -90,7 +90,11 @@ class ExecutorFactory:
executor_content_raw = conversation.query(
f"General rules: " + not_allowed() + chain_of_thought_optimization('python', 'executor.py'))
executor_content = self.extract_content_from_result(executor_content_raw, 'executor.py', match_single_block=True)
if executor_content == '':
executor_content_raw = conversation.query('Please add the executor code.')
executor_content = self.extract_content_from_result(
executor_content_raw, 'executor.py', match_single_block=True
)
persist_file(executor_content, os.path.join(EXECUTOR_FOLDER_v1, 'executor.py'))
print_colored('', '############# Test Executor #############', 'red')
@@ -222,6 +226,7 @@ Please provide the complete file with the exact same syntax to wrap the code.
+ f'\n\nHere is the test scenario the executor must pass:\n{test}'
+ f'Here are all the files I use:\n{all_files_string}'
+ f'\n\nThis error happens during the docker build process:\n{error}\n\n'
+ ((f'This is an error that is already fixed before:\n{error_before}\n\n') if error_before else '')
+ 'Look at exactly at the stack trace. First, think about what kind of error is this? '
'Then think about possible reasons which might have caused it. Then suggest how to '
'solve it. Output the files that need change. '
@@ -242,7 +247,7 @@ Please provide the complete file with the exact same syntax to wrap the code.
for file_name, content in file_name_to_content.items():
persist_file(content, os.path.join(next_executor_path, file_name))
error_before = error_before
error_before = error
else:
break

8
src/prompt_tasks.py
View File

@@ -34,7 +34,7 @@ def _task(task, tag_name, file_name, function_name=None):
def executor_file_task(executor_name, executor_description, test_scenario, package):
return _task(f'''
Write the executor called '{executor_name}'.
Write the executor called '{executor_name}'. The name is very important to keep.
It matches the following description: '{executor_description}'.
It will be tested with the following scenario: '{test_scenario}'.
For the implementation use the following package: '{package}'.
@@ -55,7 +55,7 @@ def test_executor_file_task(executor_name, test_scenario):
if test_scenario else ""
)
+ "Use the following import to import the executor: "
f"from executor import {executor_name} "
f"```\nfrom executor import {executor_name}\n```"
+ not_allowed()
+ "The test must not open local files. "
+ "The test must not mock a function of the executor. "
@@ -82,7 +82,8 @@ def docker_file_task():
"It is important to make sure that all libs are installed that are required by the python packages. "
"Usually libraries are installed with apt-get. "
"Be aware that the machine the docker container is running on does not have a GPU - only CPU. "
"Add the config.yml file to the Dockerfile. "
"Add the config.yml file to the Dockerfile. Note that the Dockerfile only has access to the files: "
"executor.py, requirements.txt, config.yml, test_executor.py. "
"The base image of the Dockerfile is FROM jinaai/jina:3.14.1-py39-standard. "
'The entrypoint is ENTRYPOINT ["jina", "executor", "--uses", "config.yml"]. '
'Make sure the all files are in the /workdir. '
@@ -144,4 +145,5 @@ The executor must not load data from the local file system unless it was created
The executor must not use a pre-trained model unless it is explicitly mentioned in the description.
The executor must not train a model.
The executor must not use any attribute of Document accept Document.text.
Note that the Dockerfile only has access to the files: executor.py, requirements.txt, config.yml, test_executor.py.
'''