ivsfm.Rd: an estimator table up front, and say what C2SLS returns (issue #3) - #51
Merged
Merged
Conversation
…sue #3) \value described every fit as reporting the coefficients first and carrying rho, vcov_rho and sigma_c; C2SLS reports sigma_u and sigma_v first, with NA standard errors, and carries none of the three. Pi and Sigma_xi are the least-squares values for IVCF and C2SLS. The details now sit under titled subsections, with the 1.2.0 note on rho's standard errors moved there from \value. \usage is unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014oJL17ewiahA8pWbemnqNp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This continues the issue #3 readability pass with
ivsfm.Rd. I held it until #46 merged, since #46 also edits this page'suhetentry.Corrected facts
These were measured on the page's own example (n = 600) on main.
sigma_u,sigma_v, …, onerho_per endogenous variable""IVLIML"and"IVCF"."C2SLS"reportssigma_uandsigma_vfirst (rownames(out):sigma_u sigma_v (Intercept) x1 x2), all withNAstandard errorsrho,vcov_rho,sigma_clisted as components of every fit"C2SLS"fits carry none of themPi,Sigma_xi: "reduced-form coefficient matrix / error covariance""IVLIML";"IVCF"and"C2SLS"return the least-squares values (Pi_ols,Sxx_olsinR/ivsfm.R)The example's claim that ignoring endogeneity biases x2 toward zero holds. The truth is −0.6; IVLIML gives −0.609, and the NHN
sfm()fit gives −0.466.Layout
"C2SLS"takes nouhet, per ivsfm: refuse uhet for C2SLS instead of silently ignoring it #46.No technical content was removed.
\usageis byte-identical, andtools::checkRd()is clean. The rendered text was checked withRdconv -t txt.🤖 Generated with Claude Code
https://claude.ai/code/session_014oJL17ewiahA8pWbemnqNp
Generated by Claude Code