« first day (184 days earlier)      last day (3380 days later) » 

Duga
Duga
12:00 AM
RELOAD!
[Vogel612/JavaBot] 1 commit.
 
 
10 hours later…
skiwi
skiwi
10:19 AM
hey
 
Duga
Duga
10:54 AM
> Added indentation field to all log entries.
 
Vogel612
Vogel612
@skiwi Hey
 
skiwi
skiwi
hey @Vogel612
 
Duga
Duga
> Merge pull request #15 from Vogel612/development

Injected TestingChatInterface to JavaBot's Program, fixes #14
 
Vogel612
Vogel612
argh damn you github... why do you use the --no-ff flag?
 
skiwi
skiwi
11:10 AM
@Vogel612 What's that?
 
Vogel612
Vogel612
it's a flag that will create a merge-commit, even if fast-forwarding is possible
without that flag a merge commit is only necessary when there's merge conflicts
 
skiwi
skiwi
I tend to not use merge commits if possible, any reason why you like them?
 
Duga
Duga
 
Vogel612
Vogel612
@skiwi I don't
but github does...
so when I press the merge pr button, it uses the --no-ff flag.
because whatever
 
skiwi
skiwi
Ah ok
 
skiwi
skiwi
11:26 AM
Adapting the parsers to save the indentation level is quite an annoying task
 
Duga
Duga
11:51 AM
> Added indentation level support to the parsers.
 
Duga
Duga
12:14 PM
> The merging of Vincentyification/JavaBot#41 has changed some things. The testclient needs to adapt
[Vogel612/JavaBot] Vogel612 pushed 12 commits to master
> Merge pull request #41 from Vogel612/develop

Development
[Vogel612/JavaBot] Vogel612 pushed 1 commit to develop
> Moved GameEntityLogEntry and PlayerLogEntry to their own top level classes.
 
Duga
Duga
12:51 PM
> Moved GameEntityEntryParser and PlayerEntryParser to their own top level classes.
 
skiwi
skiwi
Heh, this was unexpected
I changed the log entries, etc.
Then when running HearthMonitor nothing should have changed yet (it was all work to prepare for changes), but suddenly everything changed!
Turns out it currently parses log entries inside the ACTION_START tag individually, because the ACTION_START tag is not being parsed, so it reads them as individual entries
 
Duga
Duga
> Fixed bug with usage of ChatInterface
- StackExchangeChat is no more closed when getting out of main-method
 
Vogel612
Vogel612
1:09 PM
bugged because overuse of try-with-resources ...
 
skiwi
skiwi
1:27 PM
@Vogel612 `StackExchangeChat seInterface = new StackExchangeChat();
` how does that name make sense?
 
Vogel612
Vogel612
erm... ehhh....
it's implementing ChatInterface
 
skiwi
skiwi
Why is it not just called stackExchangeChat? :)
 
Vogel612
Vogel612
because... whatever.
no idea actually
it just never struck me as wrong.
 
skiwi
skiwi
well, your code, your problems ;)
 
Simon André Forsberg
Simon André Forsberg
@skiwi @bazola I'd still like your opinion on Add a license issue: github.com/Cardshifter/Cardshifter/issues/149 (@Vogel612 is free to suggest a license as well, even though he hasn't contributed to the prroject)
@skiwi remind me to say that to you in the future
 
skiwi
skiwi
1:37 PM
@SimonAndréForsberg There's a difference between pointing out mistakes in someone's code and actively asking for help..
 
Simon André Forsberg
Simon André Forsberg
true that
but still, funny comment ;)
 
Duga
Duga
> I personally find the MIT license to be not restrictive enough (especially concerning copyleft clause and attribution). IMO picking MIT is somewhat equivalent to dedicating things to public domain (also known as: CC0). I think that's not quite the plan here, so I'd advise against MIT ;)
 
Vogel612
Vogel612
@skiwi feel free to make a PR when it's bugging you ;)
2
 
skiwi
skiwi
@Vogel612 I think I'll refrain from that :p
@SimonAndréForsberg Yeah... I won't dare to say that in a company for sure!
@SimonAndréForsberg I know about the license... but don't really know what one is the best to use, still inexperienced in that
 
Duga
Duga
1:54 PM
> Merge pull request #42 from Vogel612/develop

Fixed bug with usage of ChatInterface
> Created TimerCommand, see #29

This is a working state for the timer command. Unfortunately the Improvements made for #23 are not in the working state's master. Merging #28 would greatly improve the code's quality.
[Vogel612/JavaBot] Vogel612 deleted branch 29_timer-command
 
skiwi
skiwi
What are your thoughts on the license @SimonAndréForsberg?
 
Simon André Forsberg
Simon André Forsberg
@skiwi I'm not fully educated on Apache License, but I think @Vogel612 is right about avoiding MIT for this.
 
skiwi
skiwi
Also, do you have a clue if it's possible to release all code to the public domain (say we put it all on CR eventually), yet have our whole application licensed under Apache/BSD?
 
Simon André Forsberg
Simon André Forsberg
@skiwi it is possible to dual license, yes. But putting on CR is not the same as putting it in the public domain.
 
skiwi
skiwi
2:15 PM
Isn't it similar?
 
Simon André Forsberg
Simon André Forsberg
nope
CC-BY-SA.
Attribution required
 
Phrancis
Phrancis
2:50 PM
Hello
 
Marc-Andre
Marc-Andre
hey @Phrancis!
 
Phrancis
Phrancis
How is it going?
 
Marc-Andre
Marc-Andre
good and you ?
 
Vogel612
Vogel612
@SimonAndréForsberg and copyleft
 
Phrancis
Phrancis
Not bad for a Monday morning. Pretty cold right now, you guys getting ready to storm up in QC too?
 
Vogel612
Vogel612
2:52 PM
@skiwi you can compare CC-BY-SA with CC0 (which is quasi-public domain) on creativecommons.org/licenses
 
Marc-Andre
Marc-Andre
@Phrancis It's so cold...
 
Phrancis
Phrancis
@Marc-Andre On the bright side, I got Spring Tool Suite installed and hooked up to our repo. I just don't really know how to use it, so I may need a bit of help getting started :D
 
Marc-Andre
Marc-Andre
@Phrancis No problem, tonight the GF won't be home until 8pm or 9pm
 
Phrancis
Phrancis
Ah ok that's perfect, I should be home by about 6:30
 
Duga
Duga
> Removed differentiation between InteractionBot and DefaultBot in Program
- Bots are now maintained in a Set in Program.
> Moved Command Binding to DefaultBot Instantiation
> Moved javadoc Command to CoreBotCommands.
- JavaDocAccessor is now embedded to the CoreBotCommands
- Program is no more responsible for Javadocs
> Changed Command calling: removed case-sensitivity
- also slightly simplified the filter by moving the extraction of the
command-name to outside of the stream
> Added Command Info text to a few commands
> Implemented message reply functionality, fixes #17
- PrintUtils now can "format" text as a reply given a String and a ChatMessage
- DefaultBot now replies to processed messages
> Fixed Bug in command processing
> Quoted trigger for proper removal of Regex-containing Triggers
> Updated names in PrintUtils to match naming conventions
> Patched javadoc regex to match actual syntax
> Changed DefaultBot to work on a Factory basis, this prevents
escaping of this-reference in Constructor
> Clarified listener execution lambda, used method ref for result filter
> Merge pull request #41 from Vogel612/develop

Development
> Fixed bug with usage of ChatInterface
- StackExchangeChat is no more closed when getting out of main-method
> Merge pull request #42 from Vogel612/develop

Fixed bug with usage of ChatInterface
 
Vogel612
Vogel612
3:04 PM
that should've been it...
 
Duga
Duga
 
Phrancis
Phrancis
@SimonAndréForsberg I read about Apache 2.0 it's not that long of a read and well written. It sounds similar to the Creative Commons licensing I use for music.
 
bazola
bazola
3:49 PM
@SimonAndréForsberg looking at the difference licenses, I think that apache 2.0 is probably the best fit. I'm no expert though
 
 
2 hours later…
Marc-Andre
Marc-Andre
5:58 PM
atm Cardshifter.io is more active than Cardshifter... :D
(nice job @Phrancis )
 
Phrancis
Phrancis
Hahah thanks! But, to be fair, half of them from last Friday were fixing something I screwed up because I wasn't paying attention ;D
 
Marc-Andre
Marc-Andre
@Phrancis chuuuuuuuuuuuuuuut ( is chut in english means the same thing as in french ?)
 
Phrancis
Phrancis
You would say: Shhhhhhhhhhhhhhhhhhhhhhhht!
 
bazola
bazola
6:38 PM
well, here is my bejeweled AI
0
Q: Brute Force (and not) Bejeweled AI

bazolaMy approach to solving the problem of selecting the best match on a bejeweled board is a fully object oriented approach. Some of the other relevant code can be found in my previous questions regarding bejeweled, but this question will only be about the DMMatchFinder class and associated classes....

algorithm oop game objective-c ai
 
Duga
Duga
6:54 PM
[Vogel612/JavaBot] Vogel612 pushed 2 commits to master
 
 
2 hours later…
Simon André Forsberg
Simon André Forsberg
8:45 PM
@bazola so many interesting CR questions!
 
bazola
bazola
@SimonAndréForsberg today?
 
skiwi
skiwi
I don't regret buying sub for FFXIV:ARR... my programs do regret that I bought it
 
Simon André Forsberg
Simon André Forsberg
@bazola no, perhaps not today. But there's some in my backlog that I want to take a look at.
I probably wwon't take a close look at your question though, @bazola. Me and Objective-C just don't speak the same language.
 
bazola
bazola
well that is okay, I understand that objective-c is the red headed stepchild of languages around here :)
2
 
Phrancis
Phrancis
I'm sure nhgrif will be more than happy to review it, as he normally is :)
 
Marc-Andre
Marc-Andre
9:05 PM
@SimonAndréForsberg Will probably change the webapp to Spring Boot if everything is fine well I'll be happy. Not sure how it will go. It always suppose to be easy but you know.. . Is bound to go wrong or break :P
 
Simon André Forsberg
Simon André Forsberg
@Marc-Andre Spring Boot? No idea what that is, but I trust your judgement.
I still haven't deployed the wwebapp btw... not sure what URL to put it on, or howw to put it on that URL.
 
Marc-Andre
Marc-Andre
@SimonAndréForsberg Spring Boot try to be like Rails a bit. There is a convention for everything (like webapp name, db conenctions, etc) So most of the configuration I've done would be useless with it I think.
 
Simon André Forsberg
Simon André Forsberg
never used rails either :)
 
Marc-Andre
Marc-Andre
That's next on my list after working on the website
 
Simon André Forsberg
Simon André Forsberg
9:21 PM
I was considering learning it for a whwile, but it feels like it isn't as popular now
 
Marc-Andre
Marc-Andre
still pretty big, but the thing that is popular at the moment is Javascript framework like Angular and Node.js. But rails is losing ground since now a lot of popular framework include some of the forces of rails (like Spring Boot)
 
 
1 hour later…
Phrancis
Phrancis
10:34 PM
@Marc-Andre I had a question about the io repo, specifically for images
I noticed there is a cardshifter.github.io / images folder as well as cardshifter.github.io / io.web / src / main / webapp / images
Is the 2nd one the correct one to add image files to be displayed on the site?
 
Marc-Andre
Marc-Andre
10:55 PM
second one is the correct one
the first one is the one from the merge that I didn't take time to move the files
 
Phrancis
Phrancis
Oh ok cool
 
Duga
Duga
> added stack API key to do more API requests
> added creation date to Stack Exchange Comment result
> cleaned up scanComments method a bit
> webhook parameters are now final class fields in ScheduledTasks
> using Log4j log in ScheduledTasks and logging some warnings to specific chat room
> extracted method for determining if comment is interesting
> lastCommend and fromDate set inside for-each loop
 
Simon André Forsberg
Simon André Forsberg
@Marc-Andre you here? I'm having a little Spring question
 
Marc-Andre
Marc-Andre
yep
 
Phrancis
Phrancis
TTGH bbs
 
Simon André Forsberg
Simon André Forsberg
11:04 PM
@Marc-Andre In my pom.xml for @Duga I apparently have <scope>runtime</scope> for log4j and some other logging stuff, now for some reason maven build fails on my server computer, so I can't deploy. Is there a reason whwy it is <scope>runtime</scope> or should I remove that and just use <scope>compile</scope> (the default)
I will test
 
Marc-Andre
Marc-Andre
yeah I would say go for compile since isn't runtime when you launch via maven ?
 
Simon André Forsberg
Simon André Forsberg
isn't runtime that it is expected to be provided by the runtime platform? (by Tomcat 8 in this case)
 
Marc-Andre
Marc-Andre
Ok so if runtime is provided by tomcat 8, is log4j loaded in the tomcat ?
 
Simon André Forsberg
Simon André Forsberg
no clue
 
Duga
Duga
Monking!
 
Simon André Forsberg
Simon André Forsberg
11:15 PM
@Duga's back now so at least it worked by removing the <scope>
 
Marc-Andre
Marc-Andre
yeah the things with runtime is that you now need to provide it in a way. I guess that the problem was that you didn't have log4j in the tomcat lib directory or something like that
 
Simon André Forsberg
Simon André Forsberg
I guess so
feels like it's a better idea to provide it at compile time
 
Marc-Andre
Marc-Andre
yeah Maven will set the library in the classpath and everything will be better :D
Maven provide things you don't want to use since it will punish you :P
 
Phrancis
Phrancis
11:37 PM
Hey @Marc-Andre I'm home now, going for a quick smoke then I'll be good. Do you have Skype? May be easier to talk through it especially for me because I only have one screen here
 
Marc-Andre
Marc-Andre
yeah sure ! But I need to go make supper brb in like 30 minutes
 
Phrancis
Phrancis
OK cool
Just add me whenever, search for Francis Veilleux-Gaboury, should come right up
 

« first day (184 days earlier)      last day (3380 days later) »