These are chat archives for anderspitman/dice-game

1st
Apr 2016
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 00:10
Cloning the repo = still on your repo?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 00:17
Google says I should fork and then clone!
:+1:
Nguyen Dao
@ngdao
Apr 01 2016 01:25
how do I fix my the pull request then?
I forked it
but there's no difference now
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 01:30
Did your pull request get merged?
Anders Pitman
@anderspitman
Apr 01 2016 01:34
Yeah it's already merged. No worries. In the future you'll push to hehe876/dice-game
Nguyen Dao
@ngdao
Apr 01 2016 02:16
I really think we should meet up with the TA tomorrow tho. I'll go alone if no one else can. Just tell me if we're ready?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 02:32
What are his hours?
Anders Pitman
@anderspitman
Apr 01 2016 02:47
Definitely hoping not to go on campus tomorrow, but I can if necessary
Nguyen Dao
@ngdao
Apr 01 2016 02:49
You don't have to. I just wanna make sure. So when we have what we think is good for submission, I'll run it by them
Anders Pitman
@anderspitman
Apr 01 2016 04:07
I think we should add some general documentation about how the game is played and what the rules are
Christiaan Schmid
@chcschmid
Apr 01 2016 06:46
I added some basic rules/goals to the README file. Sorry for all the commits. I think I was able to do the pull request correctly, however.
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:47
I'm finishing up the product backlog
Anders Pitman
@anderspitman
Apr 01 2016 06:49
Looks good. Merged. I'm impressed you remembered all the steps for doing the PR. It really started to sound like an incantation when I walked you through it
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:49
What statistic are we set on calculating?
As a player I want to view my personal statistics so that I can see my average number of rolls per win?
Anders Pitman
@anderspitman
Apr 01 2016 06:50
Something you'll all want to keep in mind is that when the /anderspitman/dice-game repo gets updated, you have to manually update yours. Typically you do this by setting it up as "upstream". See this: https://help.github.com/articles/configuring-a-remote-for-a-fork/
@mshannahnv I don't think we've nailed down exactly what stats we'll have.
We could pick a few obvious ones to document initially
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:52
I wasn't sure if the TA's will want something more specific for the product backlog or not
Right now we have "As a player I want to view my personal stat so that I can see how well I am playing." as the user story
Anders Pitman
@anderspitman
Apr 01 2016 06:54
It wouldn't hurt to split that into 3 or 4 more specific stats if someone wants to do it
you can just make something up honestly
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:54
ok
Anders Pitman
@anderspitman
Apr 01 2016 06:54
but there's really no limit to how far we could go with trying to guess what they want
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:55
One user story we forgot ( that I'm adding to the backlog) and we may want to add it to the User Stories markdown file as well is..
  1. As a player I want to choose to not roll so that I can maintain my score before going over the score of 23.
I'll just keep fiddling with the backlog.. and when/if everybody approves it.. we can update everything across the board
Anders Pitman
@anderspitman
Apr 01 2016 06:58
Do you have a PR for it yet?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:58
Will submit one very soon
Just found a markdown editor
Anders Pitman
@anderspitman
Apr 01 2016 06:59
k. keep in mind it doesn't have to be finished to submit the PR. You can submit what you have and that allows us to see it and make comments
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 06:59
Oh okay!
I'll do that real quick
Anders Pitman
@anderspitman
Apr 01 2016 07:00
just make sure when you push it's to github.com/mshannahnv/dice-game
ah I see you already created a branch there, so you should be good to go
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:07
So do I need to merge it on my own repo?
and then pull request into yours
I did a PR on mshannahnv/dice-game
Anders Pitman
@anderspitman
Apr 01 2016 07:07
no need to PR on yours
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:08
:(
Do I PR on yours?
Anders Pitman
@anderspitman
Apr 01 2016 07:08
just push the branch to yours, then PR mine. Once it gets merged into mine, you update yours by pulling from mine
you'll update your master by pulling from my master**
that's what "upstream" is for. but I can explain that later. for now just push everything you have to your branch then click the Create PR button
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:10
I think I'm confused by what "push" is
Anders Pitman
@anderspitman
Apr 01 2016 07:11
no worries
just a sec. I'll take a look at what you have
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:11
I'm using the Git Desktop and don't think that's helping
Anders Pitman
@anderspitman
Apr 01 2016 07:12
I see the backlog file you created. Did you do it on your computer or directly on the github web page?"
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:12
on my computer
Anders Pitman
@anderspitman
Apr 01 2016 07:13
k. so the process you used to copy it up to github is called pushing
on the command line you would do git push origin branch-name, or specifically git push origin product-backlog
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:14
So on Git Desktop it states "Commit to Product-Backlog" as a button
Anders Pitman
@anderspitman
Apr 01 2016 07:14
origin is the name of where you cloned the repo from, which for you should be github.com/mshannahnv/dice-game
hm it might be doing the push behind the scenes
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:15
Yeah I don't like the desktop client I don't think
:-1:
PR to yours?
Anders Pitman
@anderspitman
Apr 01 2016 07:16
I recommend learning the CLI version. Its really only about 5 commands you need to use most of the time
So yeah if this looks like the latest version, go ahead and create the PR
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:21
I'm getting an error with the PR
Request Failed
Validation Failed
Christiaan Schmid
@chcschmid
Apr 01 2016 07:22
whenever I pushed I pushed to git push origin master will it be different for everyone and their own repositories?
Anders Pitman
@anderspitman
Apr 01 2016 07:22
uh
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:22
Err
Anders Pitman
@anderspitman
Apr 01 2016 07:22
I've never seen that before
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:22
It worked
Anders Pitman
@anderspitman
Apr 01 2016 07:22
@chcschmid pretty much everything should be done on a branch, by convention
@mshannahnv what went wrong the first time?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:23
I may need to switch to command line - as I had two anderspitman/masters on my Github Desktop
so I tried to PR into the first one
Anders Pitman
@anderspitman
Apr 01 2016 07:23
hm
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:23
Validation failed
not sure
and then teh second one
was fine
Anders Pitman
@anderspitman
Apr 01 2016 07:23
weird
if you're using cygwin git is easy to use from there
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:24
I like cygwin a lot
I'll have to try it later
Anders Pitman
@anderspitman
Apr 01 2016 07:25
like I said it's not too much to learn the basics. git is extremely powerful and flexible which can make it daunting to get started, but you quickly realize you only need a few commands 99% of the time
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:26
Does the product backlog seem like a good "rank"
top being what would be programmed first
Because apparently that's all a product backlog is.. taking the user stories and ranking them
Anders Pitman
@anderspitman
Apr 01 2016 07:27
Yeah it looks good to me
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:27
Sweet
Anders Pitman
@anderspitman
Apr 01 2016 07:27
ok to merge?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:27
I think so
Anders Pitman
@anderspitman
Apr 01 2016 07:28
Done. Like I said, we can always update things as much as we want
I feel like we should probably make the use-case diagram easier to find. Maybe link the image from a markdown file
Excellent. At this point everyone has committed
@hehe876 I think we're pretty close to the point where you could go over this with the TAs and see if it's what they're looking for
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:31
Do you want me to update the user stories so that they match the backlog?
I think I added 1 story
Christiaan Schmid
@chcschmid
Apr 01 2016 07:31
I'm bored... I'll do it
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:31
haha
Anders Pitman
@anderspitman
Apr 01 2016 07:31
lol
We'll need to update the use case diagram as well
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:33
?
lol
what the...
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:34
haha why did it pop up with your face
Where is that at?!
Christiaan Schmid
@chcschmid
Apr 01 2016 07:34
interesting @mshannahnv ... didn't know you had pictures of Anders laying around in your file systems
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:34
Huh? That was Anders that posted it
hahaha
Anders Pitman
@anderspitman
Apr 01 2016 07:35
Yosemite
Christiaan Schmid
@chcschmid
Apr 01 2016 07:35
hahaha dang. my bad
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:36
What should we update regarding the diagram?
end game?
Christiaan Schmid
@chcschmid
Apr 01 2016 07:36
@anderspitman is there a way I can manually update my repository chcschmid/dice-game.git
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:36
if they want to end early
Anders Pitman
@anderspitman
Apr 01 2016 07:37
isn't it supposed to match the user stories 1-1?
I'm not sure...
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:37
You're right
I'm trying to figure out how to word it I guess
Christiaan Schmid
@chcschmid
Apr 01 2016 07:37
nvm. I saw your post from earlier about configuring a remote fork
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 07:37
"Choose not to roll"
Anders Pitman
@anderspitman
Apr 01 2016 07:37
@chcschmid yes. Set up an upstream to point at https://github.com/anderspitman/dice-game
Christiaan Schmid
@chcschmid
Apr 01 2016 07:38
muchas merci
Anders Pitman
@anderspitman
Apr 01 2016 07:39
just follow those 2 links in order and you should be set
Christiaan Schmid
@chcschmid
Apr 01 2016 07:39
Will do. I'm calling it a night. I'll add the user story tomorrow. Enjoy not having school tomorrow!
Anders Pitman
@anderspitman
Apr 01 2016 07:39
:+1:
me too. night guys
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 08:03
Updated the use case diagram and user stories
submitted PR
Please review tomorrow! :)
Nguyen Dao
@ngdao
Apr 01 2016 17:42
Do we have a backlog yet?
it's the same as user stories?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 18:10
the product backlog is under dice-game/doc
It's basically taking your user stories and ranking them
Are you still meeting with the TA?
Christiaan Schmid
@chcschmid
Apr 01 2016 18:35
This is a video that helped me with setting up upstream
Nguyen Dao
@ngdao
Apr 01 2016 21:39
He suggest adding an external actor that is in charge of storing the score
Otherwise everything else looks good
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 21:40
Thanks for going Eric!
Do we need another user story for the external actor then?
As a scorekeeper I will add the dice to determine the score?
and then do we need to add another "bubble" in the diagram
? like
"add dice"
Nguyen Dao
@ngdao
Apr 01 2016 21:51
New bubble can just say "record score"
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 21:51
ok
Nguyen Dao
@ngdao
Apr 01 2016 21:52
And another bubble can say "pull score" that will be included in the view stats
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 21:52
Do we need a line from roll dice - > record score?
Nguyen Dao
@ngdao
Apr 01 2016 21:53
Nah, those 2 can be separate
Maybe "give score" would be a better name than "pull score"
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:10
Did a PR for the use case diagram - somebody review please
Anders Pitman
@anderspitman
Apr 01 2016 22:15
Wait, what was his justification for the scorekeeper to be external?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:15
Yeah I don't really agree with that because it's not an external system - goes against everything we learned but
:/
Anders Pitman
@anderspitman
Apr 01 2016 22:16
Well it might make sense but I want to understand it. It'll affect how we design it
Did he just feel like having a single actor was too simple?
Or specifically that scorekeeping should be a separate system?
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:18
Maybe it's kinda external because we're going to store it in a txt file
Anders Pitman
@anderspitman
Apr 01 2016 22:19
Still doesn't make sense. I can't think of a logical way to code it up
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:19
  1. As a score keeper I want to set the score so that I can declare a win or loss when reaching or exceeding 23.

  2. As a score keeper I want to get the score so that I can provide the values to the leaderboard.

Does that look ok?
Anders Pitman
@anderspitman
Apr 01 2016 22:21
Hm maybe we could think of it like the scorekeeper is the only one allowed to modify the database file
and read from it
so the player client reports to the scorekeeper, and the score keeper records the data. Then the scorekeeper reads all the data, crunches the numbers, and sends a report to the player client
Even if we completely change it later I think that sounds really good on paper
open to other ideas though
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:26
PR for Stories/BL
Anders Pitman
@anderspitman
Apr 01 2016 22:28
That'll work. We should talk more about what this will actually mean for the code design though
both merged
sarahka7
@sarahka7
Apr 01 2016 22:29
Yes that sounds good - makes sense.
Nguyen Dao
@ngdao
Apr 01 2016 22:30
I think what he meant was the system is the game it self, and the scorekeeper is external because the game will be puting data into it and extracting data from it. It's not the game system.
Anders Pitman
@anderspitman
Apr 01 2016 22:32
Well that depends on how you design the game. I know lots of computer games where the game itself keeps score
This can add complexity. How's it actually going to work? Will we have to run 2 separate programs? Will the scorekeeper run like a daemon on the users computer? Or maybe run on a webserver?
Actually I kinda like the sound of that. It could sort of be like minecraft where you run a server then different clients can log in to play the game.
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:35
lololol
Sara and I were in CSE310 last night and somebody said they are planning on converting their dice game into a top down zombie shooter
Anders Pitman
@anderspitman
Apr 01 2016 22:36
good luck
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 22:37
Made me laugh
oh sorry Sarah** my sister spells it without an H
:)
I doubt they are expecting more than a text file
If you guys ever want to edit the Use Case Diagram - I made it in https://www.draw.io/ .. you can import the PNG and it will be editable
sarahka7
@sarahka7
Apr 01 2016 22:49
Lol @mshannahnv yeah, i just liked how he said it all nonchalant!
Btw what can i do to be helpful for the next deliverable? im keeping an eye on convos here, but if there is some piece i should specifically start working on over the weekend pls let me know.
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 23:40
Who wants to do the "submission" on blackboard? Just need a link to the github, right?
Nguyen Dao
@ngdao
Apr 01 2016 23:49
Yeah, most people are going all out with their game. Sound complicated as. Also he wants us to look at the game as a complete application as well. Like how the user want to start the game (like open an exe file or go to a website)
Hannah Van Den Bosch
@mshannahnv
Apr 01 2016 23:50
Do we need to put that in user stories
As a user I want to click on an executeable to play a dice game
:/
Or is that just something we put in "How to play"
Nguyen Dao
@ngdao
Apr 01 2016 23:52
We should when we decided on it. As of now, i think we only need to make them in correct format and correct relationship woth each other (i.e all the user stories belong to a use case)