You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently we have separate type WBSCInput to model user input and we are transforming it to WBSCOutput during evaluation process.
The types are largely the same. The most notable differences are WBSCOutput has text1 and text2 fields to represent text being printed out onto HTML Canvas and the extraOutput array to represent additional advances of the same player ("flat" array of 1-10 inputs is being mapped into 1-4 outputs with possible nested values).
Having the two types separately brings unnecessary (i.e. here) and even counter-intuitive (i.e. here) overhead. It should be possible to merge those structures into just one to avoid extra complexity.
The most challenging place to change should be renderAction method where the extra inputs for same player are handled recursively. The simpliest way to deal with this is to create the required structure on-the-fly in processAction before this loop.
The text was updated successfully, but these errors were encountered:
Currently we have separate type WBSCInput to model user input and we are transforming it to WBSCOutput during evaluation process.
The types are largely the same. The most notable differences are WBSCOutput has
text1
andtext2
fields to represent text being printed out onto HTML Canvas and theextraOutput
array to represent additional advances of the same player ("flat" array of 1-10 inputs is being mapped into 1-4 outputs with possible nested values).Having the two types separately brings unnecessary (i.e. here) and even counter-intuitive (i.e. here) overhead. It should be possible to merge those structures into just one to avoid extra complexity.
The most challenging place to change should be
renderAction
method where the extra inputs for same player are handled recursively. The simpliest way to deal with this is to create the required structure on-the-fly inprocessAction
before this loop.The text was updated successfully, but these errors were encountered: