adammotts commented 5 months ago

Description

Change below here

Please include a summary of the changes and the related issue. Please also include relevant motivation, context, and images! If its a frontend ticket, screenshots are important, if its a backend ticket, please add pictures of relevant postman screenshots or console printouts.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. If they are unit tests, provide the file name the tests are in. If they are not unit tests, describe how you tested the change.

Change below here. If this is a frontend MR, include screenshots

Add here

Checklist

[ ] I have performed a self-review of my code
[ ] I have reached out to another developer to review my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] New and existing unit tests pass locally with my changes

aniamisiorek commented 5 months ago

Looks good structure-wise! Check out my PR, I swapped out the short-term enum for a lookup table, which slightly affects your model.

leoRysing commented 4 months ago

make sure to resolve conflicts before you merge

GenerateNU / Algo