[Bug]: cyclic dependencies between model_info, seeding, and initial_conditions complicate type hinting

[pearsonca]

Label

bug, dependency

Priority Label

medium priority

Describe the bug/issue

We cannot easily introduce type hinting for modinf arguments in seeding and initial_conditions given the cyclic dependency.

We desire to not have cyclic dependencies (this enables easier refactoring, e.g.), and so should determine a way to either remove the model_info dependency in seeding / initial_conditions or vice versa.

To Reproduce

No response

Environment, if relevant

No response

[TimothyWillard]

I bumped up the priority on this. The more I think about the more I think this cyclic dependencies issue is going to lock up development at some point (in very gnarly fashion) and I'd rather address that before it blocks us. It would be especially inconvenient if that occurred when trying to fix a high priority bug or make a submission.

[pearsonca]

makes sense to me. i suspect that cyclic dependencies are more than just a model_info issue, so we should be thinking about how to reveal the others. expanding the type-hinting coverage would probably do the trick, so maybe prioritize that?