Skip to content

NEW_FRONTEND#2

Open
longsizhuo wants to merge 37 commits into
masterfrom
dev
Open

NEW_FRONTEND#2
longsizhuo wants to merge 37 commits into
masterfrom
dev

Conversation

@longsizhuo
Copy link
Copy Markdown
Owner

Please do not merge branches. This pr is to track the development status of the dev branch.

Frrrrrrrrank and others added 30 commits January 2, 2024 12:16
add the tutorial link in bilibili and youtube
Optimize display format
modifiy -> modify
简单整理了下readme 的格式
添加了环境变量的文件
add .env file for environment variables
added packages
Add others js script
增加了langchain方式生成求职信,适配使用自定义API
田间了关于faiss-cpu的注意事项
推荐微软云版本
只发送消息给新HR,不会重复发送
增加了简化付费版说明
各位见谅,宣传一下产品
… tags by cancel the comment `time.sleep(20)`

2. `write_response.py` function `create_thread`, add timeout time to avoid stuck.

3. `write_response.py` fix: no more "【" or "】" appear in the content now
### chatgpt4 及以上运行方式
如果尝试使用更新的chatGPT则不能保持最新版本为`v1.1.1`,同时如果报错信息为`An error occurred: Error code: 400 - {'error': {'message': "The requested model 'gpt-4o-mini' cannot be used with the Assistants API in v1. Follow the migration guide to upgrade to v2: https://platform.openai.com/docs/assistants/migration.", 'type': 'invalid_request_error', 'param': 'model', 'code': 'unsupported_model'}}`

需要手动将chatgpt更新到最新版,以及更改`create_assistant`中的结构体,详细参考[迁移模型](https://platform.openai.com/docs/assistants/migration)中的描述。建议直接在[平台](https://platform.openai.com/assistants/)上手动添加最新的assist然后复制代码到`assistant.json`中最为方便
```json
{"assistant_
Update README.md, 添加了chatGPT4及以上版本的使用说明
Frrrrrrrrank and others added 7 commits October 9, 2024 15:45
Btw, in my opinion this version is still runable, the description of "该项目已下架" may lead to unnecessary misleading
@longsizhuo longsizhuo requested a review from Copilot June 11, 2025 03:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR tracks the development of the dev branch by adding a new frontend (Electron) and backend (Python gRPC) along with necessary dependencies and service definitions.

  • Updated Python dependencies to support gRPC and application data management
  • Defined protobuf service/messages and generated corresponding Python gRPC code
  • Implemented Python gRPC server and Electron-based UI with IPC and streaming

Reviewed Changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
python/requirements.txt Added grpcio, grpcio-tools, protobuf, and appdirs
python/proto/job_service.proto Defined JobService RPCs and message types
python/proto/job_service_pb2.py Generated protobuf message classes
python/proto/job_service_pb2_grpc.py Generated gRPC stub and servicer classes
python/job_service_pb2.py Generated protobuf message classes for root Python package
python/job_service_pb2_grpc.py Generated gRPC stub and servicer classes for root Python package
python/grpc_server.py Implemented Python gRPC server logic and file handling
python/generate_grpc.py Added script to regenerate gRPC code from .proto
python/functions.py Introduced OpenAIClient singleton for managing OpenAI instance
langchain_functions.py Removed unused Langchain-based implementation
electron/src/renderer/styles.css Added CSS styling for the Electron renderer UI
electron/src/renderer/index.html Created main HTML structure for Electron UI
electron/src/renderer/app.js Implemented renderer-side logic and IPC handling
electron/src/main.js Set up Electron main process, Python server spawn, and gRPC client
electron/package.json Defined Electron app metadata and dependencies
TECH.md Added extensive technical documentation
README.md Expanded user instructions and project overview
README Removed outdated plain README
Comments suppressed due to low confidence (6)

python/functions.py:26

  • The OpenAI class/function is not imported in this file, which will cause a NameError. Add the appropriate import, e.g., from openai import OpenAI.
self.client = OpenAI(api_key=api_key, base_url=base_url)

python/generate_grpc.py:6

  • The script references a ./proto directory, but your .proto files live under python/proto. Update the directory paths in this script to match the actual location.
os.makedirs('proto', exist_ok=True)

electron/src/main.js:146

  • The handler signature expects fileContent and fileName, but the renderer only sends the file content. fileName will be undefined. Either adjust the renderer to send both or update the handler to accept only one argument.
ipcMain.handle('upload-resume', async (event, fileContent, fileName) => {

electron/src/main.js:188

  • You send the event 'job-completed', but the renderer listens for 'job-complete'. These names must match for the completion handler to fire.
stream.on('end', () => {

python/grpc_server.py:149

  • The attribute current_thread is referenced but never defined in the servicer, which will raise an AttributeError when stopping a job. Store the spawned thread in self.current_thread during StartJob if needed.
if self.current_thread:

python/requirements.txt:11

  • Remove the trailing whitespace at the end of this line to keep the file clean.
appdirs==1.4.4 

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants