Skip to content
This repository was archived by the owner on Jan 3, 2018. It is now read-only.

times for shell, python, git, sql #789

Merged
merged 2 commits into from
Nov 2, 2014

Conversation

chendaniely
Copy link
Contributor

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
Copy link

rgaiacs commented Oct 8, 2014

@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
Copy link
Contributor Author

@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
Copy link

rgaiacs commented Oct 8, 2014

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
Copy link
Contributor Author

@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
Copy link

rgaiacs commented Oct 8, 2014

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

I'm OK with it.

@chendaniely
Copy link
Contributor Author

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

@rgaiacs
Copy link

rgaiacs commented Oct 8, 2014

@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
Copy link
Contributor Author

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

gvwilson pushed a commit that referenced this pull request Nov 2, 2014
times for shell, python, git, sql
@gvwilson gvwilson merged commit 81b77c3 into swcarpentry:gh-pages Nov 2, 2014
@rgaiacs
Copy link

rgaiacs commented Nov 2, 2014

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

@wking
Copy link
Contributor

wking commented Nov 4, 2014

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 swcarpentry/shell-novice#11 so we don't forget.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants