Closed lukaszett closed 8 months ago
Nice job. Could you add your name and affiliation to the README.md list please?
(see topic 3 still using an attribute to set the number).
Could you add an XML comment to the test file that says this. <!-- Use attribute rather than tag -->
or something similar.
super, thanks @lukaszett
The current implementation of
_read_topics_trecxml
is expecting the topic number to be supplied via attribute to the topic tag. However, this does not seem to be the current way TREC is formatting topics in xml. See for example the health misinfo topics:My change should not break backwards compatibility, also added a test to confirm this (see topic 3 still using an attribute to set the number).