Skip to content

fix: reduce buildScoreCache example size and clean up export_abnFit example#261

Open
LeonidasZhak wants to merge 1 commit into
furrer-lab:mainfrom
LeonidasZhak:fix/254-reduce-example-sample-size
Open

fix: reduce buildScoreCache example size and clean up export_abnFit example#261
LeonidasZhak wants to merge 1 commit into
furrer-lab:mainfrom
LeonidasZhak:fix/254-reduce-example-sample-size

Conversation

@LeonidasZhak

Copy link
Copy Markdown

Summary

This PR addresses issue #254 by making the following documentation improvements:

Changes

  1. R/build_score_cache.R (line 283): Reduced N from 1e6 to 100 in the buildScoreCache() example. 1 million observations is excessive for a simple example; 100 is sufficient to demonstrate the workflow.

  2. R/export.R (lines 152-161): Cleaned up export_abnFit() example:

    • Removed unnecessary data(ex1.dag.data) call (data is available via lazy loading)
    • Removed unnecessary library(abn) call (not needed in examples)
    • Fixed missing distributions: added b4 = "binomial" and b5 = "binomial" to match the 10 variables in ex1.dag.data
  3. man/buildScoreCache.Rd and man/export_abnFit.Rd: Updated Rd files to match roxygen source

Validation

  • Verified ex1.dag.data has 10 columns: b1, p1, g1, b2, p2, b3, g2, b4, b5, g3
  • Verified the fixed mydists list now has 10 matching entries
  • Verified the diff is small and focused (4 files, 8 insertions, 14 deletions)

Related Issues

…xample

- Reduce N from 1e6 to 100 in buildScoreCache() example (issue furrer-lab#254)
- Remove unnecessary data() call from export_abnFit() example
- Fix missing distributions in export_abnFit() example (b4, b5)
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.

1 participant