(WIP) Attempt to eliminate cyclic dependency NAR→Cycle→Decision

[ARCJ137442]

Maybe reducing cyclic dependency can make the module dependency of ONA better?

• Privatize the global variable currentTime in NAR.c - completely possible
• Supplement the parameter currentTime of Decision_ConsiderNegativeOutcomes to eliminate the cyclic dependency of NAR -> Cycle -> Decision - still WIP and needs further discussions

The only problem is here:

OpenNARS-for-Applications/src/Decision.c

Line 230 in 9954416

NAR_AddInputBelief(feedbackTerm);

Are there any good alternatives to "input beliefs"?
Pass a function pointer such like void (*InputBelief) (Term) as a parameter on Decision_Execute, Cycle_ProcessAndInferGoalEvents and Cycle_Perform?

[patham9]

Awesome that you found this, thank you!
And I will think about a solution to replacing NAR_AddInputBelief(feedbackTerm) there.

[ARCJ137442]

Awesome that you found this, thank you! And I will think about a solution to replacing NAR_AddInputBelief(feedbackTerm) there.

Thanks for your watching too!

I also have some extra information that I hope will help us.

That's my dependancy analyze of ONA using Graphviz dot:

(generated on 2024-09-21, .svg vectorgraph)

And its Graphviz code can be seen and previewed here.
(2 cyclic dependencies: Narsese.h ⇄ Variable.h, Decision.h→NAR.h→Cycle.h)

Perhaps it can give us a macro overview of module dependencies in ONA :)