Skip to content

Conversation

@Cbampeta
Copy link
Collaborator

@Cbampeta Cbampeta commented Dec 4, 2025

Rajout d'un system de main ultra puissant qui lance tout les autres programmes qui ont était transformé en classe (tous ou a peu prés) et qui gere l'affichage de l'écran et rajout d'un system de streaming de la camera sur une adresse http://192.168.1.10:8000/stream.mjpg lorsque elle est lancé ainsi que une classe programme dont hérité toute les autres classe : ainsi que rajout d'un remote control depuis le lan et aussi un moyen d'éteindre la pi depuis les boutons.

Cbampeta and others added 30 commits November 1, 2025 16:54
… modif de la classe camera donc en l'etat possible probleme avec les fonctions liées
@Danura30082 Danura30082 self-requested a review December 16, 2025 10:39
Copy link
Member

@Danura30082 Danura30082 left a comment

Choose a reason for hiding this comment

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

J'ai l'impression que tu n'as pas compris ce que tu fait sur certain truc.
Il y a une multiplication enorme de fichier dans scripts et src/HL. Il faudrais faire le menage et simplifer la structure du truc. N'hesite pas aussi a faire un scheme pour montre comment les fichier interagise entre eux

Remarque global:
utilise logging, il ne devrait pas avoir de print qui traine sauf exeption
utilise des fonction main et des block if name == "main":

Copy link
Member

Choose a reason for hiding this comment

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

pourquoi preciser une partie et pas tout? Pas sure qu'on precis ici car il faudrais le tenir a jour en plus du pyproject.tml

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

je voulais préciser car se sont des dépendance qu'il faut installer pas sur uv. le reste est dans le uv sync --rpi

To Outdated
Copy link
Member

Choose a reason for hiding this comment

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

C'est quoi ce fichier

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

aucune idée (il disparait)

Copy link
Member

Choose a reason for hiding this comment

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

C'est quoi la logique derriere avoir celui la dans sont propre dossier?

def envoie_donnee(Voiture): #si utilisation de la voiture directement
print("lancement de l'i2c")
import smbus
import struct
Copy link
Member

Choose a reason for hiding this comment

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

Moyen fan des import ici mais ca ce defend

data = struct.pack('<ff', float(vitesse), float(direction))
bus.write_i2c_block_data(SLAVE_ADDRESS, 0, list(data))
def envoie_donnee(Voiture): #si utilisation de la voiture directement
print("lancement de l'i2c")
Copy link
Member

Choose a reason for hiding this comment

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

use logging

"""Initialize the car's components."""
self.vitesse_milimetres_s = 0 # Speed in millimeters per second
self.angle_degre = 0 # Steering angle in degrees
self.vitesse_d = 0 # Speed in millimeters per second
Copy link
Member

Choose a reason for hiding this comment

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

pourquoi vitesse_d?

@@ -73,9 +71,9 @@ def _initialize_tof():
_initialize_ai()

# Initialize Lidar
_initialize_lidar()
Copy link
Member

Choose a reason for hiding this comment

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

Why no more init lidar and camera?

src/HL/Car.py Outdated


class Ai_Programme():
Copy link
Member

Choose a reason for hiding this comment

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

a quoi sert cette class?

@@ -180,7 +195,9 @@ def main(self):
bp2 = Button("GPIO6")
try:
Schumacher = Driver(128, 128)
GR86 = Car(Schumacher)
GR86 = Car(Schumacher,None,None)
GR86._initialize_camera()
Copy link
Member

Choose a reason for hiding this comment

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

pourquoi les initialiser ici et pas dans _init?

global State
print("Tune Prop PWM")
State=0
def tune_dir_pwm():
Copy link
Member

Choose a reason for hiding this comment

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

comme la direction n'est pas asservie pourvoire tune les pwm de propolustion reste pertiant

Copy link
Member

@Danura30082 Danura30082 left a comment

Choose a reason for hiding this comment

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

Harmonize code base to english

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.

4 participants