Splitting the Factory (or alternatively just renaming it)

[uli-f]

The RInChIGeneratorFactory hands out three different objects: RinchiGenerator, RinchiToReaction and RinchiDecomposition.

This factory produces so much more than just 'RInChIGenerator'! 😃

I would like to either

1. introduce different factories for the three type of object or just
2. rename to factory to something more general (RInChIFactory? RInChIProcessingFactory? ...?)

Personally, I prefer Option 1, but I would very much like to hear your opinion here.

[ntk73]

I followed here the same pattern from cdk-inchi where they use only one factory.

[uli-f]

Thank you for the clarification!

I would like to leave this issue open for now and put the question to the CDK maintainers when this branch is ready for them to have a look. I am happy to put the work in if changes are necessary, this issue serves just as a reminder.

I am happy for you to apply any kind of tag (question, PR question, ...) to this and any other issues that we (or maybe that should read I 😃) want to clarify with the CDK maintainers.