With the merge, we've got some complex dictionary inputs and outputs. It'd be good for us to document these so we don't forget them later.
As a side note, I feel like the reason we're having to use all these dictionaries and such is because we're still trying to keep all database interactions isolated in the ReaderWriter. But since we're integrating the pipeline into the application, I think we could just do database interactions wherever it's needed. Of course, this is going to take a lot of code changes, so I'd rather push this off until we've got the pipeline functioning.
With the merge, we've got some complex dictionary inputs and outputs. It'd be good for us to document these so we don't forget them later.
As a side note, I feel like the reason we're having to use all these dictionaries and such is because we're still trying to keep all database interactions isolated in the ReaderWriter. But since we're integrating the pipeline into the application, I think we could just do database interactions wherever it's needed. Of course, this is going to take a lot of code changes, so I'd rather push this off until we've got the pipeline functioning.