logisim-evolution/logisim-evolution

Add the entries FpgaFreqPreDivider and FpgaFreqPreMultiplier in all the other fpga_*.properties files.

Chiusa

#1712 aperta il 5 apr 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Java (505 fork)batch import
enhancementgood first issuehelp wantedpri std

Metriche repository

Star
 (3695 stelle)
Metriche merge PR
 (Merge medio 3g 18h) (47 PR mergiate in 30 g)

Descrizione

@chuckb : Thanks a lot for your contribution, I just forgot to ask you to add the entries FpgaFreqPreDivider and FpgaFreqPreMultiplier as # ==> FpgaFreqPreDivider = and # ==> FpgaFreqPreMultiplier = in all the other fpga_*.properties files. Could you please make a separate PR to fix this? Thanks!

Well, here is a question. I do not view them as FPGA properties, since that are not fixed per circuit. The base clock on the board is a fixed value; the technology is a fixed value. But the multiplier/divider are factors more akin to circuit parameters, but only apply during synthesis. So, where should I store them, if at all?

Originally posted by @chuckb in https://github.com/logisim-evolution/logisim-evolution/issues/1702#issuecomment-1497505412

Guida contributor

Direzione di ricerca
Il problema deriva da una precedente PR (#1702) in cui il maintainer ha richiesto l'aggiunta di FpgaFreqPreDivider e FpgaFreqPreMultiplier a tutti i file fpga *.properties. Tuttavia, il contributor si chiede se questi valori appartengano alle proprietà o ai parametri del circuito. Per risolvere, esamina i file fpga *.properties esistenti (ad esempio fpga board.properties) e comprendi l'uso previsto di questi campi. Controlla la discussione nell'issue collegata e qualsiasi commento correlato dei maintainer. Quindi decidi la posizione di archiviazione appropriata e aggiorna tutti i file di conseguenza.
Tech stack
java
Dominio
desktoptooling
Tipo issue
Funzionalità
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.
In attesa del maintainer
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
Basic knowledge of Java properties filesFamiliarity with logisim evolution's FPGA configuration
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60