CleverRaven/Cataclysm-DDA

Charge removal side effects 1: tiny items

Aperta

#67.324 aperta il 31 lug 2023

Vedi su GitHubGuida contributor
 (0 commenti) (2 reazioni) (0 assegnatari)C++ (3991 fork)batch import
(S2 - Confirmed)Help WantedItems / Item Actions / Item Qualities

Metriche repository

Star
 (9264 stelle)
Metriche merge PR
 (Merge medio 6g 5h) (292 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la logica di arrotondamento delle cariche degli oggetti e controlla gli elementi nel gist fornito per correggere le loro statistiche.
Tech stack
cpp
Dominio
desktop
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65