Skip to content

feat(cmd): auto-convert electric current intensity (mA/A)#3374

Closed
zoic21 wants to merge 1 commit into
developfrom
feat/auto-convert-current-intensity
Closed

feat(cmd): auto-convert electric current intensity (mA/A)#3374
zoic21 wants to merge 1 commit into
developfrom
feat/auto-convert-current-intensity

Conversation

@zoic21

@zoic21 zoic21 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Résumé

Ajoute la conversion automatique d'unité pour l'intensité électrique : une valeur exprimée en mA est désormais mise à l'échelle en A par cmd::autoValueArray(), comme c'est déjà le cas pour la puissance (W), l'énergie (Wh), la fréquence (Hz), etc.

Détail

Nouvelle entrée dans la table $_unite_conversion de core/class/cmd.class.php :

'mA' => array(1000, 'mA', 'A'),

Le facteur 1000 et la liste d'unités suivent exactement le format existant. Exemple de rendu : 1500 mA1.5 A.

Format retenu conformément à la discussion de l'issue ('mA' => array(1000, 'mA', 'A')) ; kA volontairement écarté car peu usuel en domotique.

Closes #2429

Add a unit conversion entry for electric current so that values
expressed in mA are automatically scaled to A by autoValueArray(),
matching the existing behaviour for power, energy, frequency, etc.

Closes #2429
@zoic21 zoic21 requested review from Mips2648 and Salvialf June 15, 2026 09:16

@Mips2648 Mips2648 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Est-ce vraiment pertinent (je pose juste la question) de le faire dans ce sens?
l'unité de "base" c'est l'ampère ("A"), miliampère c'est un préfix

@Salvialf

Copy link
Copy Markdown
Contributor

Oui je suis d'accord @Mips2648, après coup j'ai aussi un doute sur le véritable intérêt de cette issue. De mémoire je l'avais ouverte suite à des retours sur le forum concernant les conversions automatiques mais j'ai peut être mal compris ou trop extrapolé.

@zoic21 zoic21 closed this Jun 15, 2026
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.

Auto convert electric current intensity (A)

3 participants