These are chat archives for gwydirsam/DickGrayson

14th
Apr 2015
Martin Fracker, Jr.
@Towerism
Apr 14 2015 00:13
awesome thanks sam
Sam Gwydir
@gwydirsam
Apr 14 2015 00:13
wait that's not a good choice
That's what I said spoke too soon
Martin Fracker, Jr.
@Towerism
Apr 14 2015 00:13
So libbmp is not good enough?
Also what kind of progress report
Sam Gwydir
@gwydirsam
Apr 14 2015 00:14
Seemed like it had issues
Just choose what you think is best, after all you need to work with it
Rafael Moreno
@ralphie9224
Apr 14 2015 00:27
i'm in dur
Sam Gwydir
@gwydirsam
Apr 14 2015 00:27
cool
one sec and I'll push
Rafael Moreno
@ralphie9224
Apr 14 2015 00:27
what does dgprime do?
Sam Gwydir
@gwydirsam
Apr 14 2015 00:28
it should be called dgrandomprime
but it's the class that does what your card is
Rafael Moreno
@ralphie9224
Apr 14 2015 00:28
what a pro
Sam Gwydir
@gwydirsam
Apr 14 2015 00:29
do git checkout feature/dgprime
so we need some tests in test/dgcrypto/dgcrypto-test.cc
and the work is in include/dgcrypto/dgrandomprime.{hh,cc}
pull again I pushed the rename
Rafael Moreno
@ralphie9224
Apr 14 2015 00:32
ok i'm pulling. by tests you meeeeaaaannnnnn?
Sam Gwydir
@gwydirsam
Apr 14 2015 00:33
TDD
we write all the tests for what our code /should/ do then we write the code to pass those tests
hit me up when you can skype--it'll be easier
Rafael Moreno
@ralphie9224
Apr 14 2015 00:34
ok yea let's do that. what's ur skype?
Sam Gwydir
@gwydirsam
Apr 14 2015 00:35
gwydirsam
Rafael Moreno
@ralphie9224
Apr 14 2015 00:35
it's like why did i even ask
Sam Gwydir
@gwydirsam
Apr 14 2015 00:36
haha
I changed everything to that fairly recently
Rafael Moreno
@ralphie9224
Apr 14 2015 00:38
smart. I sent you a request
o looky
Sam Gwydir
@gwydirsam
Apr 14 2015 02:34
Whats' the word on the progress report?
Martin Fracker, Jr.
@Towerism
Apr 14 2015 04:46
sam i installed git flow
what is supposed to be my next release branch
at least for DickGrayson repo
Martin Fracker, Jr.
@Towerism
Apr 14 2015 04:54
what should be production release branch. And what should be the next release branch
Sam Gwydir
@gwydirsam
Apr 14 2015 04:55
master is release branch
Martin Fracker, Jr.
@Towerism
Apr 14 2015 04:55
oh okay and i guess develop is next release branch
Sam Gwydir
@gwydirsam
Apr 14 2015 04:55
oh you're setting up git flow?
use the defaults
Sam Gwydir
@gwydirsam
Apr 14 2015 05:02
I guess production is master and develop is next release
but I just use the defaults
Sam Gwydir
@gwydirsam
Apr 14 2015 06:11
yo martin -- work in a feature branch
ah it looks like that's what you did
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:26
yea I fucked up and set upstream to develop
Sam Gwydir
@gwydirsam
Apr 14 2015 06:26
on git-flow?
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:26
no not on git flow
should i just let git flow handle everything
Sam Gwydir
@gwydirsam
Apr 14 2015 06:26
oh like on the branch
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:26
I kind of wanted to push to remote
Sam Gwydir
@gwydirsam
Apr 14 2015 06:26
I'm not sure to what extent it handles things
but if it can, yeah
ah yeah
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:27
would you happen to know how to make git flow push to remote without merging?
Sam Gwydir
@gwydirsam
Apr 14 2015 06:27
I'm not sure of the command
it merges on push?
I think you're attempting to do something weird
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:27
well im pretty sure
that
Sam Gwydir
@gwydirsam
Apr 14 2015 06:27
do cat .git/config
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:28
git flow feature finish stego will merge
will merge
Sam Gwydir
@gwydirsam
Apr 14 2015 06:28
oh yeah
I generally don't finish them
I set up a pull request
then merge on github once everyone is happy
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:29
does git flow do pull requests?
Sam Gwydir
@gwydirsam
Apr 14 2015 06:29
don't think so
you can push the branches though
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:29
how do i do git pull requests?
lol nvm ill just google
Sam Gwydir
@gwydirsam
Apr 14 2015 06:29
you we're on the right track before I think with pushing, but you push to just origin not origin/develop or something like that
I'm still interested to see cat .git/config
I'm also doing a renaming of directories to avoid the confusion you had
src -> bin, include -> lib
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:31
oh so where do i put headers that I want to access in the tests then?
here's cat
Sam Gwydir
@gwydirsam
Apr 14 2015 06:31
well now, `include /lib-name/{.hh,.cc}
all together
src is just for main.cc
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:32
[branch "feature/stego"]
        remote = origin
        merge = refs/heads/develop
oh okay that is what im currently doing. I just need to rename h to hh
Sam Gwydir
@gwydirsam
Apr 14 2015 06:32
yeah change develop to feature/stego
but stego is probably too large of afeature
name it something like your card name
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:33
how do i change a branch name
Sam Gwydir
@gwydirsam
Apr 14 2015 06:33
it's not worth it at this point haha
if you had emacs....haha
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:34
okay I'll name my next feature branch accordingly
I'll get it eventually. I'm still getting used to vim :P
Sam Gwydir
@gwydirsam
Apr 14 2015 06:34
see you in several years
so I'm thinking of just having everyone add their files to the cmakelists in the directory they are working in
I'm cleaning them up in feature/cleanup
to make it a bit easier
but other than that it's not really that difficult
tests on the other hand look insanely complicated but arent
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:39
CMakeLists aren't that hard. People only have to worry about the source list
I mean adding new sources to CMakeLists isn' that hard
Sam Gwydir
@gwydirsam
Apr 14 2015 06:39
checkout test/CMakeLists.txt
and tremble
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:40
haha oh yea I can see that now
Sam Gwydir
@gwydirsam
Apr 14 2015 06:40
so that EXPECT_GT(8u was there for a reason
The reason no longer really stands
but gotta be careful with that stuff
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:40
I was thinking that there was just one source list but I can also why that wouldn't be athing
what expect_gt?
Sam Gwydir
@gwydirsam
Apr 14 2015 06:41
in one of your commits you changed something in a test
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:42
I did? I didn't mean to.
Sam Gwydir
@gwydirsam
Apr 14 2015 06:42
also all of our library headers can be accessed with <lib-name/lib-name.hh>
you did a revert?
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:42
Oh yea I was in a panic when I accidentally pushed to develop
Sam Gwydir
@gwydirsam
Apr 14 2015 06:42
hahaha
it's cool
be careful with that stuff
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:43
I will. Also do you mean I can just do <lib-name/lib-name.hh> whenever I need a header in lib-name library?
Sam Gwydir
@gwydirsam
Apr 14 2015 06:43
yeah
in cleanup I'm pushing some directions though
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:44
Sweet. Maybe now syntastic will stfu
Sam Gwydir
@gwydirsam
Apr 14 2015 06:44
they'll be in readme.md in each directory
so like the lib rules are you should have <lib-name/lib-name.hh> just include all the other headers in the directory if theres more than one
but if it's a one header library, that's ok
but probably following the just include rule is a good choice no matter
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:45
Oh so I have to make the stego-crypt header
That makes sense I just originally assumed there was some voodoo magic going on
Sam Gwydir
@gwydirsam
Apr 14 2015 06:46
you give me far too much credit
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:46
Lol I really do
So for stego-crypt
Sam Gwydir
@gwydirsam
Apr 14 2015 06:47
actually now that I think about it, yeah no matter what it'll be named correctly, just do what I said anyway haha
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:47
It would be <stego-crypt/stego-crypt.hh> or <stego-crypt-lib/stego-crypt-lib.hh
Sam Gwydir
@gwydirsam
Apr 14 2015 06:48
ah..
yeah for now with -lib
though it shouldn't be that way
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:48
okay
Sam Gwydir
@gwydirsam
Apr 14 2015 06:48
I just need to do the cmake magic that makes the binary and lib of the same name not conflict
but that magic...comes with consequences...
only for me..it's just a pain though
Martin Fracker, Jr.
@Towerism
Apr 14 2015 06:52
haha I think this is alright
Sam Gwydir
@gwydirsam
Apr 14 2015 06:53
we might be able to do without the dir in the name, i'm not sure
like just <lib.hh>
Sam Gwydir
@gwydirsam
Apr 14 2015 07:08
merged current work in feature/dgprime into develop because it had some travis fixes
not ideal -- but now all tests run on seperate instances which is nice
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:36
so bmp format is so bad. There isn't a goddamn standard
Sam Gwydir
@gwydirsam
Apr 14 2015 07:36
yeah
that's the issue
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:36
Gonna have to find some cross platform way to deal with bmp
Sam Gwydir
@gwydirsam
Apr 14 2015 07:37
screw cross-platform
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:37
well
it's not like we are gonna compile images for unix platforms
Sam Gwydir
@gwydirsam
Apr 14 2015 07:37
Oh I meant the library side
yeah the bmp has to be readable
but -- some image viewers can be pretty forgiving
I'm still thinking we write out own
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:38
oh i see, well it's because in particular there is no standard for the bmp header
Sam Gwydir
@gwydirsam
Apr 14 2015 07:38
yeah we need to do some reading on how programs generally deal witht hat
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:38
which is where some useful info about the bmp would be but the offsets are specific to the platform of the bmp
Sam Gwydir
@gwydirsam
Apr 14 2015 07:39
the windows version is the most wide-spread i believe
I think they defined it originally
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:39
In that case I can continue on with just assuming that we only support windows bmp, since I also found out that is actually how I have currently implemented it
lol awesome
Sam Gwydir
@gwydirsam
Apr 14 2015 07:40
yeah just make sure they open on a reasonable amount of viewers
preferably browsers
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:40
Alrighty
Sam Gwydir
@gwydirsam
Apr 14 2015 07:40
cool
I'm going to sleep
did you see what's changed in develop
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:41
Oh no I actually can check the header to see if the bmp is windows format
also checkout my namespacing in dgcrypto
I guess actually in stego-crypt-lib you don't need the namespacing
because it's specific to stego-crypt
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:43
What does that mean?
Sam Gwydir
@gwydirsam
Apr 14 2015 07:43
note if youre stuff will be used in both stego-crypt and stego-attack you should put it in dgimg
so it runs all the tests debug and build versions seperately now
so if the build fails
it tells you which tests failed, not just the whole thing failed
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:44
ah okay so what do you mean about the namespacing thing?
Sam Gwydir
@gwydirsam
Apr 14 2015 07:44
now if they all go red...you done fucked up
so like in dgcrypt I have
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:44
that I can just say <stego-crypt.hh>?
Sam Gwydir
@gwydirsam
Apr 14 2015 07:44
namespace DG {
namespace Crypto {
class RandomInteger : public Integer {
so when I need a randominteger the long way is:
DG::Crypto::RandomInteger
which blows
but at the bottom of the file I have

//// aliases
// namespace alias
namespace dgcrypto = DG::Crypto;
// type alias
using dgrint = DG::Crypto::RandomInteger;
so you can use dgrint for randominteger and dgcrypto just the namespace in general
don't worry about it for now...it can be a pain
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:46
oh okay, well I just realized that I should but the bmp stuff in namespace img or something like that huh
And put everything in namespace DG
right>
?
Sam Gwydir
@gwydirsam
Apr 14 2015 07:46
only if it's in dg<lib-name>
otherwise it's specific to that binary
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:46
so for bmp stuff
Sam Gwydir
@gwydirsam
Apr 14 2015 07:46
so no namespace is ok
I have one set up called dgimg
but migrate later
for now, it's only specific to your thing
Martin Fracker, Jr.
@Towerism
Apr 14 2015 07:47
Okay
Well goodnight then
Sam Gwydir
@gwydirsam
Apr 14 2015 07:47
goodnight
Martin Fracker, Jr.
@Towerism
Apr 14 2015 10:57
I am up way too damn late.
But I realized that I have to push in order for travis to build. So All of the bmp stuff so far is covered by tests, but there is only proof of TDD for one of my tests, since I didn't push after I wrote the other failing tests.
Sam Gwydir
@gwydirsam
Apr 14 2015 16:28
I think it's cool if you just commit then push later, we'll have to see
Martin Fracker, Jr.
@Towerism
Apr 14 2015 18:14
Well I checked to see what Travis had been building and it looked like that it wasn't building my commits indivually, only the pushes
Sam Gwydir
@gwydirsam
Apr 14 2015 18:14
okay, so I guess we need to emphasize pushing
basically theres two important ones: here's my failing tests and then here's them passing
I guess the second one is going to be several commits, but you don't really need to stress pushing that one
it's the failing version
that is important
can you check out an error for me?
it's a linking issue
and I cannot figure it out.
checkout feature/coveralls-newlayout
Sam Gwydir
@gwydirsam
Apr 14 2015 18:21
it works on build... I'm now more confused
Sam Gwydir
@gwydirsam
Apr 14 2015 18:42
online scrum today tes?
yes*
also martin, if chris didn't tell you -- you /dont/ want to install gtest system-wide, that's what I was trying to tell you in lab -- the cmake does it correctly for us.
Martin Fracker, Jr.
@Towerism
Apr 14 2015 19:18
Chris told me
Sam Gwydir
@gwydirsam
Apr 14 2015 19:18
haha yeah
it was funny because he asked me first, and I was like oh you specifically dont install it system-wide and he broke out laughing
Martin Fracker, Jr.
@Towerism
Apr 14 2015 19:19
Was that today?
Sam Gwydir
@gwydirsam
Apr 14 2015 19:19
last night
Martin Fracker, Jr.
@Towerism
Apr 14 2015 19:19
Lol yea we were arguing about it
Sam Gwydir
@gwydirsam
Apr 14 2015 19:21
yeah don't take what I say as dogma...dogma's are rarely a good thing
Martin Fracker, Jr.
@Towerism
Apr 14 2015 19:23
Well I saw what Google had to say about it and it convinced me.
Sam Gwydir
@gwydirsam
Apr 14 2015 19:24
Yeah it's a weird library because it's project specific
Martin Fracker, Jr.
@Towerism
Apr 14 2015 19:31
yea I understand now why it's so difficult to install it systemwide
Sam Gwydir
@gwydirsam
Apr 14 2015 19:38
are we scrummin online?
Christopher Findeisen
@Atrus7
Apr 14 2015 19:56
Skype
Sam Gwydir
@gwydirsam
Apr 14 2015 19:59
kyle what's your skype?
@bluyam
Kyle Wilson
@bluyam
Apr 14 2015 20:00
Hmmm
I will make one
Gitter wouldn't load any messages until right just now so I thought we were meeting in personnn
Sam Gwydir
@gwydirsam
Apr 14 2015 20:02
Ah
weird
Rafael Moreno
@ralphie9224
Apr 14 2015 20:02
I'm in annex scrum ready I believe
Sam Gwydir
@gwydirsam
Apr 14 2015 20:02
adding you
Christopher Findeisen
@Atrus7
Apr 14 2015 20:03
Added you Ralf
@bluyam , do you have a handle yet?
Kyle Wilson
@bluyam
Apr 14 2015 20:07
Ugh fuck everything about skype
I'm trying to use Facebook to sign up but then it thinks I don't have a password and it's just sksskdhadbds
Kyle Wilson
@bluyam
Apr 14 2015 20:12
I just created two different accounts and it says Skype can't connect for both so just keep on scrummin without me
Martin Fracker, Jr.
@Towerism
Apr 14 2015 20:13
eno1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet 10.217.1.179  netmask 255.255.0.0  broadcast 10.217.255.255
        inet6 fe80::76d4:35ff:fe8b:2602  prefixlen 64  scopeid 0x20<link>
        ether 74:d4:35:8b:26:02  txqueuelen 1000  (Ethernet)
        RX packets 186409  bytes 189029697 (180.2 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 81633  bytes 8382344 (7.9 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
        device interrupt 20  memory 0xb0900000-b0920000  

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet 127.0.0.1  netmask 255.0.0.0
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 0  (Local Loopback)
        RX packets 536  bytes 17067 (16.6 KiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 536  bytes 17067 (16.6 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
Sam Gwydir
@gwydirsam
Apr 14 2015 20:13
@blue
Christopher Findeisen
@Atrus7
Apr 14 2015 20:13
do a phone call.
Sam Gwydir
@gwydirsam
Apr 14 2015 20:13
@bluyam can you make one without connecting to facebook?
Christopher Findeisen
@Atrus7
Apr 14 2015 20:14
We're going to do a phone call. Not having someone there is not an option
Kyle Wilson
@bluyam
Apr 14 2015 20:15
Yeah bubs I did but it's bein a lil shit
Martin Fracker, Jr.
@Towerism
Apr 14 2015 23:19
Hey Sam, can you help me with my dns problem
Sam Gwydir
@gwydirsam
Apr 14 2015 23:26
why yes I can
skype?
also fyi if anyone looks on travis and sees that the build times are like 3 hours now, it's because it sums the build time for each of 16 build configs, even though they're mostly done in parallel
real-time its around 10 min
@Towerism ready when you are
Martin Fracker, Jr.
@Towerism
Apr 14 2015 23:36
I'm ready