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

Design and Implement a _Study_ Object #36

Open
iqis opened this issue Jun 24, 2019 · 14 comments
Open

Design and Implement a _Study_ Object #36

iqis opened this issue Jun 24, 2019 · 14 comments
Assignees
Labels
enhancement New feature or request

Comments

@iqis
Copy link
Contributor

iqis commented Jun 24, 2019

... composed of many psyphr_workbook objects, with subject/activity identification inferred from file structure of workbooks (See #21).

Able to:

  • Generate high-level summaries/visualization across subjects/activities
  • Output all data in desired format to the file system
    • save_study()
  • Helper function to saveRDS() (?)

S3 object, class name: psyphr_study

generics:

  • print.psyphr_study()
  • summary.psyphr_study()
  • ...

Use a control file in YAML or DCF.

@iqis iqis self-assigned this Jun 24, 2019
@iqis iqis added the enhancement New feature or request label Jun 24, 2019
@wendtke wendtke self-assigned this Jun 25, 2019
@iqis
Copy link
Contributor Author

iqis commented Jun 25, 2019

compose_study():

  • compose multiple books into a study

read_study():

  • read multiple books from a folder
  • compose_study()

@wendtke
Copy link
Owner

wendtke commented Jun 25, 2019

Would users be confused by read vs. compose? Maybe compile_study() might be more intuitive?

@iqis
Copy link
Contributor Author

iqis commented Jun 25, 2019 via email

@iqis
Copy link
Contributor Author

iqis commented Jun 25, 2019 via email

@wendtke
Copy link
Owner

wendtke commented Jun 25, 2019

"assemble"
"collate"
"arrange"
"amass"

@iqis
Copy link
Contributor Author

iqis commented Jun 25, 2019 via email

@wendtke
Copy link
Owner

wendtke commented Jun 25, 2019

collate_study() might work well, if we think users might think "compile" refers to compiling a program.

@wendtke
Copy link
Owner

wendtke commented Jul 20, 2019

@iqis Is this issue unique compared to #37, #38, and #43? Can we consolidate?

@iqis
Copy link
Contributor Author

iqis commented Jul 20, 2019 via email

@wendtke
Copy link
Owner

wendtke commented Jul 20, 2019

Unique or consolidate? Can you combine the 4 issues the way you want it?

@iqis
Copy link
Contributor Author

iqis commented Jul 22, 2019

let's consolidate #36, #38, #43. I think #37 kinda deserves its own issue.

@wendtke
Copy link
Owner

wendtke commented Jul 23, 2019

Sounds good. Can you consolidate?

@iqis
Copy link
Contributor Author

iqis commented Jul 23, 2019

I need more time to go through them all, as there are also dev branches currently associated with each.

@wendtke
Copy link
Owner

wendtke commented Jul 23, 2019

Of course -- I meant on your own timeline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants