Togli dal percorso caldo la rilettura dell'intero storico
Sei modifiche chieste in questa sessione, di cui una pesa piu' di tutte le altre. CompletedAuctionsStore rileggeva e rideserializzava l'intero file a ogni chiamata: misurati 72 ms su 5876 record, 3,9 MB. Append pagava quel prezzo piu' la riserializzazione e la riscrittura completa per cambiare un record solo, 120 ms in tutto, e la chiamata arriva da un gestore di eventi di MainWindow, quindi dal thread dell'interfaccia, che restava fermo a ogni asta conclusa. Il recupero delle puntate dei vincitori lo faceva in un ciclo: 4675 aste erano 7,2 minuti di solo rimescolamento JSON e 17,8 GB scritti sul disco. Adesso c'e' una copia in memoria invalidata sulla data di modifica del file, cosi' resta giusta anche se il file viene sostituito da fuori (ripristino di un backup), e un AppendMany che applica molti record con un salvataggio solo, con un indice per id perche' il caso per cui esiste non torni quadratico. Il recupero salva a blocchi di cinquanta e deposita comunque quello che ha in mano quando esce, anche per annullamento: le richieste gia' fatte sono state pagate al server. Misurato sullo stesso file: 95 volte piu' veloce, 7,2 minuti diventano 4,5 secondi, 17,8 GB diventano 0,36 GB. Le altre cinque: - Il setup si chiama AutoBidder_<versione>.exe. - I pulsanti di stop sono rossi, sia quello globale sia quelli per asta nella griglia: erano gli unici tre senza colore mentre avvia e osserva ce l'avevano. - La pulizia dello storico sa togliere le aste seguite solo in parte. Non serve nessun campo nuovo: il prezzo finale in centesimi e' il totale delle puntate dell'asta, quindi il rapporto con i reset osservati dice quanta asta si e' vista, e la regola vale anche sullo storico gia' raccolto. Verificata sui dati veri: su 5876 aste nessuna supera copertura 1,02, che e' quanto ci si aspetta se l'identita' e' giusta. - La griglia ricicla i contenitori di riga invece di ricrearli. - Il costo puntata gia' esisteva col predefinito giusto: al suggerimento e' stato aggiunto perche' 0,20 e' il numero da tenere. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -46,6 +46,17 @@
|
||||
ToolTip="Senza il «Compra Subito» non si può calcolare il risparmio, ma il prezzo e le puntate restano validi."
|
||||
Checked="Option_Changed" Unchecked="Option_Changed"/>
|
||||
|
||||
<StackPanel Orientation="Horizontal" Margin="0,10,0,0">
|
||||
<CheckBox x:Name="ChkCoverage" VerticalAlignment="Center"
|
||||
Content="Aste seguite per meno del"
|
||||
ToolTip="Toglie le aste agganciate a metà. Sono il caso peggiore: i numeri sono veri ma parziali, quindi sembrano buone e abbassano le medie senza dare segnali. Quanta asta si è vista si ricava dal prezzo finale, senza bisogno di alcun dato in più: ogni puntata vale un centesimo, quindi il prezzo in centesimi è il totale delle puntate dell'asta, e i reset osservati dicono quanti se ne sono visti. Sul tuo storico, su 5876 aste: al 75% ne resterebbero circa 78 su 100, al 90% circa 64, al 95% circa 54, al 99% circa 34."
|
||||
Checked="Option_Changed" Unchecked="Option_Changed"/>
|
||||
<TextBox x:Name="CoverageBox" Text="90" Width="50" Margin="8,0"
|
||||
VerticalAlignment="Center" TextChanged="Option_Changed"/>
|
||||
<TextBlock Text="% della loro durata" VerticalAlignment="Center"
|
||||
Foreground="{DynamicResource Brush.Text}"/>
|
||||
</StackPanel>
|
||||
|
||||
<StackPanel Orientation="Horizontal" Margin="0,10,0,0">
|
||||
<CheckBox x:Name="ChkOlderThan" VerticalAlignment="Center"
|
||||
Content="Aste chiuse da più di"
|
||||
|
||||
@@ -45,6 +45,17 @@ namespace AutoBidder.Dialogs
|
||||
RemoveWithoutValue = ChkNoValue.IsChecked == true
|
||||
};
|
||||
|
||||
// Una soglia illeggibile o fuori scala non deve diventare "togli tutto":
|
||||
// il filtro resta spento finché il numero non ha senso.
|
||||
if (ChkCoverage.IsChecked == true &&
|
||||
double.TryParse(CoverageBox.Text.Trim().Replace(',', '.'),
|
||||
System.Globalization.NumberStyles.Float,
|
||||
System.Globalization.CultureInfo.InvariantCulture, out var percento) &&
|
||||
percento is > 0 and <= 100)
|
||||
{
|
||||
options.MinObservedCoverage = percento / 100.0;
|
||||
}
|
||||
|
||||
if (ChkOlderThan.IsChecked == true &&
|
||||
int.TryParse(DaysBox.Text.Trim(), out var days) && days > 0)
|
||||
{
|
||||
|
||||
Reference in New Issue
Block a user