search

swcarpentry / DEPRECATED-bc

DEPRECATED: This repository is now frozen - please see individual lesson repositories.
Other
299 stars 383 forks source link

times for shell, python, git, sql #789

Closed chendaniely closed 9 years ago

chendaniely commented 9 years ago

this is a PR for #689 I have not gotten around to the R data in my e-mails yet, but please review if this is the proper location and information you want presented to the instructors.

@gvwilson Are we doing PR's against master or gh-pages? I can't find/remember where the final verdict as to when we are making the master/gh-pages change

rgaiacs commented 9 years ago

@chendaniely Thanks for this PR. Maybe we should only present the mean at the README?

I'm thinking that maybe we could provide the raw data somewhere. I know this have some privacy issues but maybe we could find some workaround.

chendaniely commented 9 years ago

@r-gaia-cs I can remove the names and location and put identifying info in a separate table. The current information I have is very messy, but I can post those if you want (they won't mean more than what's in this PR though...)

I ended up reporting all those basic stats, mostly because the sample size is so small.
For Git, the range was quite far from each other.
Also, I only got 1 SQL response, I did not want to give any false impression that SQL is always being taught.

I did not report mode, becuase Git's mode would be 1 hour. Because @DamienIrving taught 3 times :p

rgaiacs commented 9 years ago

I can remove the names and location and put identifying info in a separate table.

I was thinking that we can starting doing it for all workshops but we need a good way to insert new entries.

The current information I have is very messy, but I can post those if you want (they won't mean more than what's in this PR though...)

You can do it later.

chendaniely commented 9 years ago

@r-gaia-cs: Actually, I can drop the other stats and just keep n and median.

I'm only against mean, because of the low N and high range of values. Plus medians won't give you weird decimal values. But either seem to give enough sense as to what's going on :)

Other than that, is this where you want the information posted? At least for the section by section level. I have to add survey information for the individual topics, and I plan to add that to the top of each individual lesson

rgaiacs commented 9 years ago

@r-gaia-cs: Actually, I can drop the other stats and just keep n and median.

I'm OK with it.

chendaniely commented 9 years ago

@r-gaia-cs sorry, which branch does this PR into? Master or gh-pages?

rgaiacs commented 9 years ago

@r-gaia-cs sorry, which branch does this PR into? Master or gh-pages?

gh-pages. If you create the pull request to master we can merge it and I later I will create one pull request from master to gh-pages.

chendaniely commented 9 years ago

@r-gaia-cs @gvwilson can this be merged?

rgaiacs commented 9 years ago

@wking This PR changed novice/shell/README.txt. How do you prefer to port it to https://github.com/wking/swc-modular-shell?

wking commented 9 years ago

On Sun, Nov 02, 2014 at 01:55:19PM -0800, Raniere Silva wrote:

This PR changed novice/shell/README.txt. How do you prefer to port it to https://github.com/wking/swc-modular-shell?

We should be able to just cherry-pick it over, and then just take extra care when we integrate this back into bc (or maybe we're dumping bc and starting with a new base for integrating the stand-alone lessons?). I've opened wking/swc-modular-shell#11 so we don't forget.