benbhansen-stats / propertee

Prognostic Regression Offsets with Propagation of ERrors, for Treatment Effect Estimation (IES R305D210029).
https://benbhansen-stats.github.io/propertee/
Other
2 stars 0 forks source link

"design" ==> "specification" #190

Open benthestatistician opened 1 week ago

benthestatistician commented 1 week ago

Let's update our nomenclature as follows.

Rationale:

josherrickson commented 1 week ago

@jwasserman2 @xinhew0708 I plan to start tackling this this afternoon. This is going to be a seismic change to the repo, so if anyone has any outstanding branches they'd like to get merged in today first, please speak up now so I can hold off.

josherrickson commented 1 week ago

Completed

Outstanding

Technical details

I did four substitutions

  1. _design to _spec
  2. Design to StudySpecification
  3. design to specification
  4. des to spec
  5. (Technically a few more smaller ones like d <- rct_spec... is not spec <- rct_spec.)

The first three were done programmatically. It should cause no issues with code, but it could have caused issues with text (e.g. a sentenced reading "Design your study carefully." would now say "StudySpecification your study carefully."). E.g.

find R/ -type f -exec sed -i '' s/_design/_spec/g {} +

The fourth and below were done semi-manually; I gave each file a brief spot check to ensure des only appeared where I wanted to make a change, e.g.

grep spec tests/testthat/test.SandwichLayer.R

taking note of any lines I saw where I didn't want to make the change, the running the appropriate find command, followed by manually fixing the noted lines.

josherrickson commented 1 week ago

@benthestatistician I just noticed that we had previously rd_design. Your list above specifies renaming rdd_design to rdd_spec. Do you want rdd_spec or rd_spec?

benthestatistician commented 1 week ago

Let's stick with "rd_", ie go to rd_spec. (Sorry for the oversight.)

josherrickson commented 6 days ago

This felt large enough for a minor bump, just to be able to point to a clear change point in language.

jwasserman2 commented 2 days ago

Some of the language in the intro-to-propertee vignette now reads somewhat awkwardly with the one-to-one replacement (brought to my attention by some questions from Cassandra). Flagging that if we continue to use that as the primary document for users, I think we'll want to revise some of the phrasing.

Also, it looks like the MI vignette isn't populating on the website after the recent push. Can you check that out, Josh?

josherrickson commented 20 hours ago

@jwasserman2 see my 2nd to last comment in #70 - It's on my list to address when I get a chance.