OpenChemistry/avogadrolibs

Export to MP4 should test for "avconv" in path

Aperta

#282 aperta il 18 giu 2018

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (1 assegnatario)C++ (232 fork)auto 404
bughacktoberfest

Metriche repository

Star
 (662 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

https://github.com/OpenChemistry/avogadrolibs/blob/da6ebb9f0e39c73ca861001e390d13e6ed3e6cb5/avogadro/qtplugins/playertool/playertool.cpp#L249

This requires avconv to be installed in the system. If avconv is not installed, this particular snippet still executes, but no mp4 file is saved and there is no warning or error thrown saying that avconv is not found.

https://github.com/OpenChemistry/avogadrolibs/blob/da6ebb9f0e39c73ca861001e390d13e6ed3e6cb5/avogadro/qtplugins/playertool/playertool.cpp#L232 In Windows, the png export itself isn't taking place. avconv execution will also presumably fail I guess. Should we resort to including libraries for these in the build?

Guida contributor

Direzione di ricerca
Ispeziona il codice intorno alla riga 249 in avogadro/qtplugins/playertool/playertool.cpp per capire come viene invocato avconv. Ricerca come verificare se avconv è disponibile sul sistema (ad esempio, usando QStandardPaths o QProcess per cercare nel PATH). Determina il modo migliore per avvisare l'utente o generare un errore quando avconv manca, e considera le differenze specifiche di Windows.
Tech stack
cpp
Dominio
desktop devaudio video rtc
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.
Attiva
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.
75