CleverRaven/Cataclysm-DDA

Charge removal side effects 1: tiny items

Offen

#67.324 geöffnet am 31.07.2023

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C++ (3.991 Forks)batch import
(S2 - Confirmed)Help WantedItems / Item Actions / Item Qualities

Repository-Metriken

Stars
 (9.264 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 5h) (292 gemergte PRs in 30 T)

Beschreibung

Describe the bug

A few items had really tiny charges smaller than 1ml. Because we have no unit smaller than that, all of them were rounded up to 1ml and most likely could use an audit to check their stats.

The list of affected items can be found here: https://gist.github.com/mqrause/dc234737bfd5c4309aa4dc496d17bd53#file-oversized_charge_removal-txt

Attach save file

n/a

Steps to reproduce

n/a

Expected behavior

n/a

Screenshots

No response

Versions and configuration

6088293934801590f503ff9d5d961e708af564ff

Additional context

No response

Contributor Guide

Research-Richtung
Untersuche die Rundungslogik für Gegenstandsladungen und überprüfe die Elemente im bereitgestellten Gist, um ihre Statistiken zu korrigieren.
Tech Stack
cpp
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65