Skip to content

Update stata do files to use new analysis data and allow for do files to run in isolation from Stata - #41

Merged
NKeleher merged 13 commits into
mainfrom
feat/new-data
Jan 21, 2026
Merged

Update stata do files to use new analysis data and allow for do files to run in isolation from Stata#41
NKeleher merged 13 commits into
mainfrom
feat/new-data

Conversation

@NKeleher

@NKeleher NKeleher commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Pull Request Summary 🚀

What does this PR do? 📝

Update the data/raw/sample_data.csv that @krojas01 created in #29 (I fixed the duplicates for ease of running just stata-run but am including a data/raw/sample_data_with_duplicates.csv for use in training and demostrations.

I also made several fixes to the do files in do_files so that each do file can run in isolation either from Stata or using nbstata in VS Code.

Why is this change needed? 🤔

Allow for more elaborate data pipeline demonstrations/training exercises and make it easier for users to work with individual do files from Stata or from VS Code.

How was this implemented? 🛠️

  • added the sample_data.csv that Karen created
  • added a STANDALONE INITIALIZATION section in each do file that allows each do file to run independently if they don't want to run the do file via do_files/00_run.do

How to test or reproduce? 🧪

  • run the full pipeline using just stata-run
  • run some of the individual do files independently

Screenshots (if applicable) 📷

Checklist ✅

  • I have run and tested my changes locally
  • I have limited this PR to less than 1000 lines of code change (if not, explain why)
  • I have updated/added tests to cover my changes (if applicable)
  • I have updated/added requirements to cover my changes (if applicable)
  • I have run linting and formatting on any code changes (if applicable)
  • I have updated the documentation (README, etc.) accordingly

Reviewer Emoji Legend

:code: Meaning
😃👍💯 :smiley: :+1: :100: I like this...

...and I want the author to know it! This is a way to highlight positive parts of a code review.
⭐⭐⭐ :star: :star: :star: Important to fix before PR can be approved...

And I am providing reasons why it needs to be addressed as well as suggested improvements.
⭐⭐ :star: :star: Important to fix but non-blocking for PR approval...

And I am providing suggestions where it could be improved either in this PR or later.
:star: Give this some thought but non-blocking for PR approval...

...and consider this a suggestion, not a requirement.
:question: I have a question.

This should be a fully formed question with sufficient information and context that requires a response.
📝 :memo: This is an explanatory note, fun fact, or relevant commentary that does not require any action.
:pick: This is a nitpick.

This does not require any changes and is often better left unsaid. This may include stylistic, formatting, or organization suggestions and should likely be prevented/enforced by linting if they really matter.
♻️ :recycle: Suggestion for refactoring.

Should include enough context to be actionable and not be considered a nitpick.

@NKeleher
NKeleher marked this pull request as ready for review January 20, 2026 19:49
@claude

claude Bot commented Jan 20, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Comment thread Justfile
@NKeleher
NKeleher merged commit f98959d into main Jan 21, 2026
2 checks passed
@NKeleher
NKeleher deleted the feat/new-data branch January 21, 2026 02:06
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