Remove useless codes and add lint.

.github/workflows/build.yml

@@ -31,7 +31,7 @@ jobs:
           path: /home/runner/.local/share/tts
           key: ${{ runner.os }}-tts
 
-      - name: Generate Prime Numbers
+      - name: Cache TTS models.
         if: steps.cache-tts.outputs.cache-hit != 'true'
         run: rm -rf /home/runner/.local/share/tts
 
@@ -43,5 +43,8 @@ jobs:
           echo "$VIRTUAL_ENV/bin" >> $GITHUB_PATH
           echo "VIRTUAL_ENV=$VIRTUAL_ENV" >> $GITHUB_ENV
 
+      - name: lint
+        run: pyright src
+
       - name: e2e
         run: echo y | slide-to-video --model ${{ matrix.model }} --slide example/slide.pdf --script example/script.txt --voice-sample-path example/sample.mp3 --output-dir output-${{ matrix.model }}

pyproject.toml

@@ -38,6 +38,7 @@ build-backend = "hatchling.build"
 managed = true
 dev-dependencies = [
     "pyinstrument>=4.6.2",
+    "pyright>=1.1.366",
 ]
 
 [tool.hatch.metadata]

src/script/__init__.py

@@ -5,7 +5,6 @@
 
 
 app = typer.Typer()
-app_dataset = typer.Typer()
 
 
 @app.command()
@@ -57,16 +56,5 @@ def generate(
     slide_to_video(project_config=project_config)
 
 
-@app_dataset.command()
-def dataset_generate(audios_path: str, output_path: str):
-    from slide_to_video.dataset import generate_dataset
-
-    generate_dataset(audios_path, output_path)
-
-
 def main():
     app()
-
-
-def dataset():
-    app_dataset()

src/slide_to_video/project.py

@@ -47,7 +47,9 @@ def content(self):
         with open(self.path, "r") as f:
             return f.read()
 
-    def __eq__(self, other: Item):
+    def __eq__(self, other: object):
+        if not isinstance(other, Item):
+            return False
         return self.md5sum == other.md5sum and self.extra == other.extra
 
     def cache(self):
@@ -297,7 +299,9 @@ def get_scripts(self, filter_cached=False):
         return [item.content for item in self.script_items]
 
     def build(self):
-        tts_engine = create_engine(self.config.get("model"), self.config)
+        model = self.config.get("model")
+        assert model
+        tts_engine = create_engine(model, self.config)
         lock = None
         if not tts_engine.parallizable():
             manager = Manager()