THUDM / CogVideo

text and image to video generation: CogVideoX (2024) and CogVideo (ICLR 2023)
Apache License 2.0
7.88k stars 737 forks source link

Pylint alerts corrections as part of intervention experiment #330

Open evidencebp opened 2 weeks ago

evidencebp commented 2 weeks ago

Feature request / 功能建议

I'd like to conduct a software engineering experiment regarding the benefit of Pylint alerts removal.The experiment is described here. In the experiments, Pylint is used with some specific alerts, files are selected for intervention and control.After the interventions are done, one can wait and examine the results. I'm asking your approval for conducting the interventions in your repositories.The interventions are expected to benefit the project and will submitted in PR for approval.

These are the planed interventions.

Motivation / 动机

Academic research in order to understand the benefit of Pylint alerts removal. Code quality improvement in this project.

Your contribution / 您的贡献

Currently there is no PR. I'll create it with the interventions once you approve.

evidencebp commented 1 week ago

@zRzRzRzRzRzRzR , I see that you wrote finetune\train_cogvideox_lora.py, one of the file with alert. May I do the interventions?