changed trader interface to take the commodity map rather than an exchange context
shorted request_by_commodity and related names