gazebosim/sdformat

Should validate the contents of <include>

Aperta

#661 aperta il 9 ago 2021

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (1 assegnatario)C++ (122 fork)auto 404
bughelp wanted

Metriche repository

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

Descrizione

Currently, sdf::readXml is not called on the <include> element, so its contents are not validated according to the SDFormat spec. Any unknown elements are silently ignored by the parser except when using the interface API. This is different behavior than when unknown elements are encountered elsewhere (e.g. in ).

For example:

<sdf version='1.8'>
  <world name='default'>
     <include>
        <uri>test_model</uri>
        <name>override_model_name</name>
         <pose>4 5 6 0 0 0</pose>
         <foo>bar</foo>
      </include>
   </world>
</sdf>

The element <foo> should trigger a warning/error depending on the configuration of the parser.

Guida contributor

Direzione di ricerca
Esamina la gestione dell'elemento <include> nel parser SDFormat per garantire che gli elementi sconosciuti attivino errori o avvisi di convalida, simile ad altri elementi come <model>. Guarda il percorso del codice per <include> e confrontalo con <model> per identificare le differenze.
Tech stack
cpp
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++XML parsing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60