Coding Projects and Duga Heaven :)

skiwi

06:52

Monking

@SimonAndréForsberg Yes the mod needs to test itself. But I'm not testing the mod here, I'm testing the mod-loader

Simon André Forsberg

07:36

10 hours ago, by Simon André Forsberg

If you need a jar for it, make a jar once and put it into the test/resources folder. Don't do it through Maven.

skiwi

@SimonAndréForsberg Isn't that ugly?

I believe it can be done through a cardshifter-core-parent without harming anything else

Simon André Forsberg

@skiwi Absolutely not. The JAR is a resource, and if you want to test the mod loader you need to provide it with a resource.

I think making it through a cardshifter-core-parent is far uglier. And unnecessary.

There won't be any further changes to that JAR, right? So why add a dependency to it?

skiwi

But we still want the cardshifter-testmod-java in our repository?

This where using Maven is being a shame... All I actually want is to first build cardshifter-testmod-java and then move the JAR to the testresources folder, but as many have said, doing anything non-Maven is hard in Maven

Simon André Forsberg

@skiwi I think we could go either way there. Or we could add a second repository for it, it might be good to have an example mod for further modders.

skiwi

@SimonAndréForsberg That's something else though... I think it goes both ways from here. On one hand we want to test the cardshifter-core to see if a most minimalistic mod works... On the other hand, we want to have reasonable mods implementing the same game in all supporting languages (Java, Lua, etc.) and also run integration tests on those

Simon André Forsberg

07:45

@skiwi So I guess we could have both a second and a third repository for it?

skiwi

@SimonAndréForsberg It's an option yes... But keep in mind that now it involves manual copying of the JAR to the test resources in case of an update

Which is why we are using a Build system in the first place, to have it automatic

Simon André Forsberg

@skiwi Yes, but I believe copying the JAR manually to test/resources is the best way to go.

IMO it is a bad idea to add a mod, that should be purely stand-alone, to the build system.

skiwi

I'm not so sure, it really belongs to the testing part of the core itself

Simon André Forsberg

But once you write that mod, there is no reason to change it.

It is a resource.

skiwi

I think we need a 3rd opinion on this ;)

Simon André Forsberg

08:10

Yes, it starts to feel like we're fighting over duck season and rabbit season...

skiwi

There must be Maven experts around... somewhere

Perhaps hidden in the deeps of Programmers.SE

Duga

08:26

[Cardshifter/Cardshifter] Zomis opened issue #46: All creatures should have 'Trample'

> The game designed was intended so that if a 4/5 attacks a 2/1, 3 damage (4 - 1) should go through and hit the player.

[Cardshifter/Cardshifter] Zomis added label ready to issue #46: All creatures should have 'Trample'

Duga

09:00

[Cardshifter/Cardshifter] Zomis pushed commit 573d94e6 to ais

> Made ECSGame final to indicate that it should not be extended

[Cardshifter/Cardshifter] Zomis pushed commit 81321dd0 to ais

> Added toString and getActionName to SpecificActionSystem

[Cardshifter/Cardshifter] Zomis pushed commit 2e6f1e97 to ais

> Added getResource to ResourceRetriever

[Cardshifter/Cardshifter] Zomis pushed commit 3d74066a to ais

> Added toString to UseCostSystem

[Cardshifter/Cardshifter] build #232 for commit 3d74066a on branch ais passed

skiwi

@SimonAndréForsberg Are you sure that those commits need to be on the branch ais?

Simon André Forsberg

@skiwi Technically, they don't really belong here no, but I had to do them when debugging things related to AIs.

skiwi

No problem

Simon André Forsberg

@bazola Does not depend on these changes, and neither do you, so I don't think there will be any merge conflicts or other problems, and I think we'll merge pretty soon anyway.

skiwi

Sounds good

Duga

09:08

[Cardshifter/Cardshifter] Zomis pushed commit 94fe5dec to ais

> Reduced logging in EventExecutor for now

[Cardshifter/Cardshifter] Zomis pushed commit e7977917 to ais

> Added requirement that the performer of action is the current player

[Cardshifter/Cardshifter] Zomis pushed commit f428d9f6 to ais

> Added AIComponent and AISystem

[Cardshifter/Cardshifter] build #233 for commit f428d9f6 on branch ais passed

[Cardshifter/Cardshifter] Zomis pushed commit bcc54e5a to ais

> standalone client performs End Turn action, prepared for using AISystem

[Cardshifter/Cardshifter] Zomis pushed commit d10d49e3 to ais

> Server now uses AISystem to perform the AI in a separate thread

[Cardshifter/Cardshifter] build #234 for commit d10d49e3 on branch ais passed

[Cardshifter/Cardshifter] Zomis pushed 12 commits to ecs-utils

[Cardshifter/Cardshifter] Zomis pushed commit a1981525 to ecs-utils

> Added a minimalistic SimpleGame

[Cardshifter/Cardshifter] build #235 for commit a1981525 on branch ecs-utils passed

Duga

09:35

Ping: Practicality beats purity.

[Zomis/AIScorers] Zomis pushed commit 5c3a72ef to master

> Converted to maven

[Zomis/AIScorers] Zomis pushed commit 4a395d9b to master

> updated gitignore

skiwi

@Duga A wild ping has apeared!

Duga

09:59

[Zomis/AIScorers] Zomis pushed commit fbddc694 to master

> Renamed TTMain to TicTacTest and made it a test

[Zomis/AIScorers] Zomis pushed commit c8313a3f to master

> Added assertion for first move

[Zomis/AIScorers] Zomis pushed commit fc918b74 to master

> Added @Before method and initializing Random to specific seed

[Zomis/AIScorers] Zomis pushed commit 04e3e426 to master

> Added more assertions, using LinkedHashMap to avoid randomness

Duga

11:10

[Cardshifter/Cardshifter] Zomis pushed commit 510be7f1 to ais

> Added AIScorer dependency

[Cardshifter/Cardshifter] Zomis pushed commit 28f34848 to ais

> Added AI that only ends turn

[Cardshifter/Cardshifter] build #236 for commit 28f34848 on branch ais passed

Simon André Forsberg

Food time, then I'll start working on some smarter AIs, and add an option to choose an AI

skiwi

11:24

@SimonAndréForsberg Doesn't the AI choose itself?

Simon André Forsberg

11:48

@skiwi choose itself? How do you mean?

skiwi

@SimonAndréForsberg Well, isn't the AI smart enough? :)

Duga

[Cardshifter/Cardshifter] Zomis opened issue #47: One launcher to choose local game or server game

> Currently the JavaFX application has two entry points. One for playing a local game and one for playing on the server. It would be better if there was only one. All that's really required is to add a button to the existing network launcher that starts a local game, with the FXMLGameController.

[Cardshifter/Cardshifter] Zomis added label enhancement to issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] Zomis added label ready to issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] Zomis removed label enhancement from issue #47: One launcher to choose local game or server game

Simon André Forsberg

@skiwi I mean that it should be possible to choose to play against a bad AI and a good AI. Currently there is no option for the user to set which AI it should play against.

skiwi

@SimonAndréForsberg Ah wait, you said AI, not a mind-reader

Duga

12:39

[Cardshifter/Cardshifter] Zomis commented on commit 333c5a12

> This doesn't actually have any effect as when there is only one action available, it is the "End Turn" action.

Duga

14:31

[Cardshifter/Cardshifter] Zomis pushed commit ea4046c0 to ais

> Added some different AI options to Server

[Cardshifter/Cardshifter] Zomis pushed commit 05bcb584 to ais

> Added some functionality for AI Medium

Simon André Forsberg

@bazola @skiwi I'd recommend trying out AI Medium. Start a server and write ai medium to create it. Then start the client and play (AIs are no longer added automatically)

Duga

[Cardshifter/Cardshifter] Zomis added label next release to issue #37: More AIs, and better AIs. Possibility to choose AI

[Cardshifter/Cardshifter] Zomis removed label in progress from issue #37: More AIs, and better AIs. Possibility to choose AI

[Cardshifter/Cardshifter] Zomis opened issue #48: Local game client menu

> When playing a local game client, it would be nice to have a menu (using the Menu class) where it could be possible to change the AI, among other things.

[Cardshifter/Cardshifter] Zomis added label enhancement to issue #48: Local game client menu

[Cardshifter/Cardshifter] Zomis added label ready to issue #48: Local game client menu

[Cardshifter/Cardshifter] build #237 for commit 05bcb584 on branch ais passed

Duga

14:49

[Cardshifter/Cardshifter] Zomis pushed commit 06e6b251 to ais

> Added functionality for when game ends. Fixes #22

[Cardshifter/Cardshifter] Zomis pushed commit 6cea7926 to ais

> Renamed TargetSet getTargets to getChosenTargets to avoid confusion

[Cardshifter/Cardshifter] Zomis removed label in progress from issue #22: GUI continues when game is over

[Cardshifter/Cardshifter] Zomis added label next release to issue #22: GUI continues when game is over

[Cardshifter/Cardshifter] build #238 for commit 6cea7926 on branch ais passed

bazola

@SimonAndréForsberg will do

Simon André Forsberg

Hey @bazola

Good that you are here, there are plenty of things for you to code ;)

bazola

looks like it :)

Duga

[Cardshifter/Cardshifter] Zomis pushed 33 commits to develop

[Cardshifter/Cardshifter] Zomis pushed commit 11a5d5cf to issue10

> Added insertTemporaryBeforeCurrent method to add a pre-game phase

[Cardshifter/Cardshifter] Zomis pushed commit 75bf14d2 to issue10

> Added Mulligan functionality to game model

[Cardshifter/Cardshifter] build #239 for commit 6cea7926 on branch develop passed

bazola

so how do I send a command to the server to do "ai medium" in the console

Duga

15:05

[Cardshifter/Cardshifter] build #240 for commit 75bf14d2 on branch issue10 errored

Simon André Forsberg

@bazola you can't send a command from the client, you can only write it on the server. Make sure that you are have pulled the changes to the ais branch. In fact, you might actually be able to do a rebase: git rebase origin/ais (to keep the git history cleaner). Or if you want to merge, that's fine: git pull origin ais

Duga

[Cardshifter/Cardshifter] Zomis pushed commit 31a63d7c to issue10

> Corrected according to recent changes for events and actions

bazola

@SimonAndréForsberg If I can't send a command to the server to start the AI of choice, how can I build button in the client to choose which AI to play against? :)

Simon André Forsberg

@bazola you can't... yet ;) You can however make a menu in the standalone-client (FXMLGameController) :)

Duga

[Cardshifter/Cardshifter] build #241 for commit 31a63d7c on branch issue10 failed

bazola

15:20

it will be painful to look at the old client now

Simon André Forsberg

lol

I know the feelings

Duga

15:36

[Cardshifter/Cardshifter] Zomis pushed commit 88b4f1be to issue10

> when currentEntity is null, all actions are no longer prevented

[Cardshifter/Cardshifter] build #242 for commit 88b4f1be on branch issue10 fixed

skiwi

16:06

@SimonAndréForsberg I'll try to fix some time for it later, need to fix osmething else now

@SimonAndréForsberg You know that remotely located servers are -for the time being- not so useful now? :)

EngieOP

19:24

[quote]test[/quote]

> test

Simon André Forsberg

@skiwi yes, I am aware. They will definitely be more useful when we have more players and/or the possibility to challenge AIs

skiwi

@SimonAndréForsberg psst, if you are bored:

3

Q: Parsing dates from an OCR application

I wrote this code to parse dates from the output of the OCR, which means that the obtained date can be literally anything, so I put some restrictions in place: Date is the the format of: field1?field2?field3, where the fields are either day, month or year and any delimiter can be used to split ...

I know the perfect thing to do!

Simon André Forsberg

@skiwi me, bored?

EngieOP

> quote

Simon André Forsberg

@EngieOP Hello

skiwi

19:29

@SimonAndréForsberg Oh... right

EngieOP

drat..noticed.
was using this room to test :P
I uh..hello

Simon André Forsberg

@EngieOP I believe there is a room somewhere that is purely for testing/spamming

EngieOP

I'm done. :)

Duga

[Cardshifter/Cardshifter] bazola added label in progress to issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] bazola removed label ready from issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] bazola assigned bazola to issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] bazola added label in progress to issue #48: Local game client menu

[Cardshifter/Cardshifter] bazola removed label ready from issue #48: Local game client menu

[Cardshifter/Cardshifter] bazola assigned bazola to issue #48: Local game client menu

[Cardshifter/Cardshifter] Zomis added label enhancement to issue #47: One launcher to choose local game or server game

[Cardshifter/Cardshifter] Zomis added label gameplay to issue #46: All creatures should have 'Trample'

[Cardshifter/Cardshifter] Zomis added label mod to issue #46: All creatures should have 'Trample'

bazola

20:14

do you know what is more fun than refactoring UI elements? everything. :)

2

skiwi

@bazola Hehe

bazola

not working on our game though, working on my strategy game.. I build this UI a long time ago and I finally decided to improve it.. many hours later, still working on it

skiwi

;)

Duga

[Cardshifter/Cardshifter] Zomis pushed 19 commits to client-server

skiwi

0

Q: The ja_JP_JP_#u-ca-japanese locale cannot be reparsed by its own pattern using the Java 8 Date and Time API

I've run across a strange issue when testing some method of mine and I seem to have been able to obtain a concrete example of my issue. I am using the ja_JP_JP_#u-ca-japanese locale not able to parse a date using it's own date pattern defined by the locale. I am wondering whether I am doing som...

java locale java-8 java-time

Meanwhile I broke Java 8.

Simon André Forsberg

20:22

@skiwi again!?

skiwi

@SimonAndréForsberg I think so...

This time it's not as obvious or bad as the last time though

So I'm not 100% sure of whether it is really broken

Duga

[Cardshifter/Cardshifter] build #243 for commit 6cea7926 on branch client-server passed

skiwi

20:34

No love fof the question yet :(

Going to bed either way

Simon André Forsberg

22:32

Bed time here as well. Lots of stuff I want to code, but I'm freakin' tired

bazola

22:42

night @SimonAndréForsberg. I will definitely work on the UI for our game tomorrow

Transcript for

Coding Projects and Duga Heaven :)