Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[old comment] #7

Closed
tingyuansen opened this issue Mar 22, 2015 · 9 comments
Closed

[old comment] #7

tingyuansen opened this issue Mar 22, 2015 · 9 comments

Comments

@tingyuansen
Copy link

[old comment]

@cccabot
Copy link
Contributor

cccabot commented Mar 23, 2015

Can you provide the snippet of code where you manually combined estimatestruct + mle_estimateparams? And what indicated that lg_estimatebn() was not performing skel.toporder()?

@tingyuansen
Copy link
Author

[old comment]

@cccabot
Copy link
Contributor

cccabot commented Mar 25, 2015

I tested your scenario and noticed that repeatedly running toporder() on a GraphSkeleton produced different topological orderings. This would explain some of your issue. I believe all the orderings were still valid, but since it's confusing behavior I pushed up a change to make the results of toporder() consistent.

@cccabot
Copy link
Contributor

cccabot commented Mar 25, 2015

In response to the second question: currently the library can calculate the likelihood of specific outcomes only for discrete Bayesian networks. Are you looking for similar functionality for linear Gaussian Bayesian networks?

@tingyuansen
Copy link
Author

[old comment]

@cccabot
Copy link
Contributor

cccabot commented Mar 25, 2015

Glad to help and thanks for the submission. We have no plans as of now to implement that functionality, but we'll keep the feature in mind, and we are happy to accept pull requests if you or others would like to contribute.

@cccabot cccabot closed this as completed Mar 25, 2015
@tingyuansen tingyuansen changed the title estimatebn != estimatestruct + mle_estimateparams [old comment] Mar 25, 2015
@tingyuansen
Copy link
Author

Hey Charlie,

Thanks! Many thanks for your suggestions, I am definitely interested to implement that, as I will need to use that in my research anyway. I am just wondering if there is any way I could be credited if I were to develop that?

I also confirmed that the discrepancy is resolved. However, I find that, as the order is fixed now, the structure learning depends on the order given in the data. In other words, if I were to rearrange my columns, I could get different fitting, some of which are better than the other. Since there is no way to evaluate the maximum likelihood in each case, I haven't found a way to compare them automatically (i.e. randomize the order and find a heuristic best fitting).

Thank you!

Cheers,
Yuan-Sen

@mikewestcp
Copy link
Contributor

Yuan-Sen,

We would be happy to credit your enhancement. We should proceed as follows:

  • fork our repo (into your personal area)
  • Make your changes. Provide tests and documentation.
  • Issue a pull-request through github
  • We will review your changes and merge
    Cheers!
    Mike

Mike West
CyberPoint Labs
direct +1 410 779 6757
mobile +1 443 520 3950

CyberPoint International

621 East Pratt Street, Suite 300

Baltimore MD 21202-3140

phone +1 410 779 6700

www.cyberpointllc.com http://www.cyberpointllc.com/

If you believe you received this e-mail in error, please notify the sender
immediately, delete the e-mail from your computer and do not copy or
disclose it to anyone else.

The information in this email constitutes the proprietary information of
Cyber Point International, LLC (DBA CyberPoint), and should be accessed only
by the individual to whom it is addressed. The information in this email and
any attachments may not be used, copied or disclosed without the consent of
CyberPoint. CyberPoint is not responsible for any damages caused by your
unauthorized use of the materials in this email.

From: Yuan-Sen Ting notifications@github.com
Reply-To: CyberPoint/libpgm
<reply+00322f49fda165789d13c3d54419940fef5f78048be0e35292cf00000001112bca3f9
2a169ce03c99a83@reply.github.com>
Date: Thursday, March 26, 2015 at 9:23 AM
To: CyberPoint/libpgm libpgm@noreply.github.com
Subject: Re: [libpgm] old comment

Hey Charlie,

Thanks! Many thanks for your suggestions, I am definitely interested to
implement that, as I will need to use that in my research anyway. I am just
wondering if there is any way I could be credited if I were to develop that?

I also confirmed that the discrepancy is resolved. However, I find that, as
the order is fixed now, the structure learning depends on the order given in
the data. In other words, if I were to rearrange my columns, I could get
different fitting, some of which are better than the other. Since there is
no way to evaluate the maximum likelihood in each case, I haven't found a
way to compare them automatically (i.e. randomize the order and find a
heuristic best fitting).

Thank you!

Cheers,
Yuan-Sen


Reply to this email directly or view it on GitHub
#7 (comment) .

@tingyuansen
Copy link
Author

Thank you, Mike for the information. Hopefully I will get back to you on
the upgrade in the near future :).

On Thu, Mar 26, 2015 at 9:26 AM, mike notifications@github.com wrote:

Yuan-Sen,

We would be happy to credit your enhancement. We should proceed as follows:

  • fork our repo (into your personal area)
  • Make your changes. Provide tests and documentation.
  • Issue a pull-request through github
  • We will review your changes and merge
    Cheers!
    Mike

Mike West
CyberPoint Labs
direct +1 410 779 6757
mobile +1 443 520 3950

CyberPoint International

621 East Pratt Street, Suite 300

Baltimore MD 21202-3140

phone +1 410 779 6700

www.cyberpointllc.com http://www.cyberpointllc.com/

If you believe you received this e-mail in error, please notify the sender
immediately, delete the e-mail from your computer and do not copy or
disclose it to anyone else.

The information in this email constitutes the proprietary information of
Cyber Point International, LLC (DBA CyberPoint), and should be accessed
only
by the individual to whom it is addressed. The information in this email
and
any attachments may not be used, copied or disclosed without the consent of
CyberPoint. CyberPoint is not responsible for any damages caused by your
unauthorized use of the materials in this email.

From: Yuan-Sen Ting notifications@github.com
Reply-To: CyberPoint/libpgm

<reply+00322f49fda165789d13c3d54419940fef5f78048be0e35292cf00000001112bca3f9
2a169ce03c99a83@reply.github.com>
Date: Thursday, March 26, 2015 at 9:23 AM
To: CyberPoint/libpgm libpgm@noreply.github.com
Subject: Re: [libpgm] old comment

Hey Charlie,

Thanks! Many thanks for your suggestions, I am definitely interested to
implement that, as I will need to use that in my research anyway. I am just
wondering if there is any way I could be credited if I were to develop
that?

I also confirmed that the discrepancy is resolved. However, I find that, as
the order is fixed now, the structure learning depends on the order given
in
the data. In other words, if I were to rearrange my columns, I could get
different fitting, some of which are better than the other. Since there is
no way to evaluate the maximum likelihood in each case, I haven't found a
way to compare them automatically (i.e. randomize the order and find a
heuristic best fitting).

Thank you!

Cheers,
Yuan-Sen


Reply to this email directly or view it on GitHub
#7 (comment) .


Reply to this email directly or view it on GitHub
#7 (comment).

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

No branches or pull requests

3 participants