wendtke / psyphr

legacy repo for R package suite for psychophysiological data; see github.com/psyphr-dev
1 stars 1 forks source link

Design and Implement a _Study_ Object #36

Open iqis opened 5 years ago

iqis commented 5 years ago

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

Able to:

generics:

Use a control file in YAML or DCF.

iqis commented 5 years ago

compose_study():

read_study():

wendtke commented 5 years ago

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

iqis commented 5 years ago

'read' and 'compose' sound as if they were opposites. "compile" is better. However, for a some people compile may smell like "compiling a program into machine code". What are some other words that mean putting things together?

iqis commented 5 years ago

"Coalesce" probably wouldn't work.

On Tue, Jun 25, 2019 at 1:59 PM Siqi Zhang iqis.gnahz@gmail.com wrote:

'read' and 'compose' sound as if they were opposites. "compile" is better. However, for a some people compile may smell like "compiling a program into machine code". What are some other words that mean putting things together?

wendtke commented 5 years ago

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

iqis commented 5 years ago

Thanks. I like "compile" and "collate"

On Tue, Jun 25, 2019 at 2:02 PM Kathleen Wendt notifications@github.com wrote:

"assemble" "collate" "arrange" "collect"

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/wendtke/psyphr/issues/36?email_source=notifications&email_token=AKE6JFRLW5R67J5DZM4IJLLP4JMS5A5CNFSM4H3CO6S2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYRC2JQ#issuecomment-505556262, or mute the thread https://github.com/notifications/unsubscribe-auth/AKE6JFWD6Q7VHWPWORFIEIDP4JMS5ANCNFSM4H3CO6SQ .

wendtke commented 5 years ago

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

wendtke commented 5 years ago

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

iqis commented 5 years ago

Yeah, I agree.

On Sat, Jul 20, 2019, 4:07 PM Kathleen Wendt notifications@github.com wrote:

@iqis https://github.com/iqis Is this issue unique compared to #38 https://github.com/wendtke/psyphr/issues/38 and #43 https://github.com/wendtke/psyphr/issues/43? Can we consolidate?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/wendtke/psyphr/issues/36?email_source=notifications&email_token=AKE6JFSJXZZALT7QYCGPWPDQANV63A5CNFSM4H3CO6S2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NVDEQ#issuecomment-513495442, or mute the thread https://github.com/notifications/unsubscribe-auth/AKE6JFX26TUNEUKQWRMXMSDQANV63ANCNFSM4H3CO6SQ .

wendtke commented 5 years ago

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

iqis commented 5 years ago

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

wendtke commented 5 years ago

Sounds good. Can you consolidate?

iqis commented 5 years ago

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

wendtke commented 5 years ago

Of course -- I meant on your own timeline.