Skip to content

Conversation

@z275748353
Copy link
Collaborator

No description provided.

@ganisback ganisback requested a review from HaiHui886 August 13, 2025 12:32
Dockerfile Outdated
# CMD ["df-server"]
CMD ["uvicorn", "data_server.main:app", "--host", "0.0.0.0", "--port", "8000"]

# # 复制并设置 entrypoint.sh 脚本的权限
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please just use english in all code comments and readm.md

Copy link
Collaborator

Choose a reason for hiding this comment

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

do not remove current license file

README.md Outdated
-e POSTGRES_USER=postgres \
-e POSTGRES_PASSWORD=postgres \
opencsg-registry.cn-beijing.cr.aliyuncs.com/opencsg_public/csghub/postgres:15.10
opencsg-registry.cn-beijing.cr.aliyuncs.com/opencsg_public/postgres
Copy link
Collaborator

Choose a reason for hiding this comment

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

Leave postgres to 15.10

Dockerfile Outdated

# # 复制并设置 entrypoint.sh 脚本的权限
# COPY entrypoint.sh /entrypoint.sh
# RUN chmod +x /entrypoint.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are those change required?


```

## Run server in development mode locally
Copy link
Collaborator

Choose a reason for hiding this comment

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

why delete those debug section?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove useless images

Copy link
Collaborator

Choose a reason for hiding this comment

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

remove

Copy link
Collaborator

Choose a reason for hiding this comment

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

remove test file

with open(readme_path, encoding='utf-8') as stream:
content = stream.read()
else:
with open(readme_path) as stream: # 在 Linux 和 macOS 下不显式指定编码
Copy link
Collaborator

Choose a reason for hiding this comment

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

注释改一下

@celery_app.task(name="collection_file_task")
def collection_file_task(task_uid: str,user_name: str,user_token: str):
"""
采集任务
Copy link
Collaborator

Choose a reason for hiding this comment

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

注释


def get_collection_task_by_uid(db_session: Session, task_uid: str):
"""
根据任务UID获取唯一的采集任务信息
Copy link
Collaborator

Choose a reason for hiding this comment

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

注释

"""
获取格式转换任务信息
Args:
db_session (Session): 数据库会话
Copy link
Collaborator

Choose a reason for hiding this comment

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

注释


def get_pipline_job_by_uid(db_session: Session, job_uid: str):
"""
根据任务UID获取唯一的pipline任务信息
Copy link
Collaborator

Choose a reason for hiding this comment

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

注释

task_uid: str
):
insert_formatity_task_log_info(task_uid,
f"转换目录:{tmp_path},源文件类型:{getFormatTypeName(from_type)},目标文件类型:{getFormatTypeName(to_type)}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

log

-v /home/redisdata:/data \
-e MONGO_INITDB_ROOT_USERNAME=root \
-e MONGO_INITDB_ROOT_PASSWORD=example \
opencsg-registry.cn-beijing.cr.aliyuncs.com/opencsghq/redis:7.2.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

mongodb 的环境变量放到了redis 来了

@ganisback ganisback merged commit a82c22b into OpenCSGs:main Aug 14, 2025
ganisback pushed a commit that referenced this pull request Aug 17, 2025
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.

3 participants