Skip to content

Add Prior knowledge for LiM #145#146

Open
yujota wants to merge 1 commit intocdt15:masterfrom
yujota:LiM-prior-knowledge
Open

Add Prior knowledge for LiM #145#146
yujota wants to merge 1 commit intocdt15:masterfrom
yujota:LiM-prior-knowledge

Conversation

@yujota
Copy link

@yujota yujota commented Aug 26, 2024

Hi there. Thank you for sharing wonderful lib.

I implemented prior knowledge for LiM, related to issue #145.

Still, I'm wondering if this implementation is theoretically ok.
I look forward to your suggestion.

Add prior knowledge option for LiM class.
Each element of the prior knowledge matrix should be 0 (unconnected) or
-1 (unknown).
Modifications to the algorithm are:
Set bounds of variables of unconnected path as 0 for L-BFGS on global
search.
Omitting paths of those to be unconnected for local search.
@sshimizu2006 sshimizu2006 requested a review from YanaZeng August 27, 2024 01:24
@YanaZeng
Copy link
Collaborator

Dear yujota,

Thanks for implementing prior knowledge for LiM. The modified codes look feasible and ok.

But please note that LiM optimizes the adjacency matrix about the directed EDGES. Element "1" in the matrix means the existence of the directed edge, other than the directed PATH.

Thank you again!
Best,
Yana

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants