Page MenuHomeWildfire Games
Feed All Stories

Tue, Oct 12

Langbart added a comment to rP25869: Allow to enable Cartography at the start of a match..

I can push a fix soon. Should this be done in a new patch?

Tue, Oct 12, 12:54 AM
Jammyjamjamman added a comment to rP25869: Allow to enable Cartography at the start of a match..

An issue has been discovered that can be observed for the first time with this changeset. See #6356

Tue, Oct 12, 12:17 AM

Mon, Oct 11

alre added a comment to D4294: smoother formation manouvres.
  1. The column thing is nice, but they move into it too quick, maybe increasing the distance for that satisfies both camps?
Mon, Oct 11, 10:09 PM
autobuild committed rP25962: [Windows] Automated build..
[Windows] Automated build.
Mon, Oct 11, 9:35 PM
Vulcan added a comment to D3249: Move parabolic range computation to rangemanager.

Successful build - Chance fights ever on the side of the prudent.

Mon, Oct 11, 9:25 PM
Freagarach added inline comments to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.
Mon, Oct 11, 8:42 PM
andy5995 updated the diff for D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

Still trying to get the value of g_MapSettings.EvenPlayerSpacing for the newly-added condition in the .js files. Still getting the same result. Always evaluates to true.

Mon, Oct 11, 8:37 PM
Freagarach added a comment to rP25953: Add accelerations in unit movement..

It looks also that formation members outrun the controller and have to move backwards every now and then?

Mon, Oct 11, 8:05 PM
Freagarach added a comment to D4294: smoother formation manouvres.

@alre Have you installed the command-line tools alongside? I have information that TortoiseSVN can't add context.
(See: https://stackoverflow.com/questions/32723100/svn-is-not-recognized-as-an-internal-or-external-command-operable-program-or/53117050#53117050.)

Mon, Oct 11, 8:04 PM
andy5995 updated the diff for D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

No changes, just regenerated the diff with 'git diff -U9999 master'

Mon, Oct 11, 7:58 PM
Freagarach edited reviewers for D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup, added: Restricted Owners Package; removed: Stan, Angen, Freagarach.
Mon, Oct 11, 7:56 PM
andy5995 updated the diff for D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

I completed removed african_plains_tg.js and mainland_tg.js. The option is now in the .json files.

Mon, Oct 11, 7:52 PM
andy5995 added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.
Mon, Oct 11, 7:43 PM
Freagarach added a comment to D3544: Regroup members at most once when formation and its members get idle.

Or even move the responsibility of keeping the correct position to the members. (And get back into formation when we become idle.)

Mon, Oct 11, 7:40 PM
Freagarach published D4302: Also do a postMove update when the average speed over last turn isn't zero for review.
Mon, Oct 11, 7:37 PM
andy5995 added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

So now I've avoided duplicating the maps, and just use 2 distinct json files, adding the setting "EvenPlayerSpacing".

Mon, Oct 11, 7:36 PM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

This should definitely be an option, especially when the other option is having almost duplicated maps. ^^

Mon, Oct 11, 7:03 PM
alllearning updated alllearning.
Mon, Oct 11, 6:19 PM
Langbart raised a concern with rP25869: Allow to enable Cartography at the start of a match..

An issue has been discovered that can be observed for the first time with this changeset. See #6356

Mon, Oct 11, 4:30 PM
Langbart added a comment to rP25958: Allow to push items to the front of the ProductionQueue..

Noticed:
(1) The sentence This item is paused.
Why is it in paranthesis? Why not add some color, for example orange. I would even put the little hand symbol in front of it, then everyone who hovers over it will see faster what it is about.

Mon, Oct 11, 3:30 PM
vladislavbelov committed rP25961: Removes mentions of legacy and unused GL calls, unifies AsFloatArray..
Removes mentions of legacy and unused GL calls, unifies AsFloatArray.
Mon, Oct 11, 2:39 PM
vladislavbelov committed rP25960: Removes direct GL calls from Atlas bandbox drawing..
Removes direct GL calls from Atlas bandbox drawing.
Mon, Oct 11, 1:31 PM
vladislavbelov closed D4300: Removes direct GL from Atlas bandbox drawing.
Mon, Oct 11, 1:31 PM
Langbart added a comment to rP25953: Add accelerations in unit movement..

I would only give this to special units like ships or battering rams, but not normal human units.

Mon, Oct 11, 11:27 AM
Langbart added a comment to rP25953: Add accelerations in unit movement..

This change makes it very cumbersome to use formations when moving units around the map. Since the formation is set to the box formation by default (rP24480), new players will be shot to pieces. At least change it to "none".


Testing it on the sceanrio map - Pathfinding Demo with 33 units (1 cavalry, 16 spearman, 16 archers).
First with rP25952: (35secs)

Mon, Oct 11, 11:08 AM
Vulcan added a comment to D3249: Move parabolic range computation to rangemanager.

Build failure - The Moirai have given mortals hearts that can endure.

Mon, Oct 11, 11:04 AM
autobuild committed rP25959: [i18n] Updated POT and PO files..
[i18n] Updated POT and PO files.
Mon, Oct 11, 9:13 AM
Stan added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

I wonder if it should/could be an option.

Mon, Oct 11, 8:54 AM
andy5995 added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

These 2 maps are from the stock mainland and African Plains map.

Mon, Oct 11, 7:14 AM
andy5995 requested review of D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.
Mon, Oct 11, 7:11 AM

Sun, Oct 10

Vulcan added a comment to D3249: Move parabolic range computation to rangemanager.

Successful build - Chance fights ever on the side of the prudent.

Sun, Oct 10, 11:19 PM
Vulcan added a comment to D3249: Move parabolic range computation to rangemanager.

Build failure - The Moirai have given mortals hearts that can endure.

Sun, Oct 10, 10:50 PM
bb updated the diff for D3249: Move parabolic range computation to rangemanager.

Fix formula

Sun, Oct 10, 10:49 PM
alre added inline comments to D4294: smoother formation manouvres.
Sun, Oct 10, 10:49 PM
alre added a comment to D4294: smoother formation manouvres.
In D4294#183089, @bb wrote:

Thanks for the patch: here is some quick review, to help you improve the patch:

Please add extra context (as stan requests), this will easy reviewing a lot.

Please add yourself to the programming.json, that way you will be in the credits.

As I see it, this patch does two things:

  1. It computes new positions when big turns and sets the rotation based on the position, instead of unit rotation.
  2. It removes the column switch behaviour.
  1. This sounds like a good idea, though it needs to be configurable what "sharp" means (it might differ from shape to shape)
  2. There surely was a good reason to add it back in the days (https://wildfiregames.com/forum/topic/13726-formations/, #13), not sure if we need to remove this.

Please try to limit the amount of unnecessary changed (moving function around and stuff). Smaller patch => quicker review

Sun, Oct 10, 10:45 PM
alre updated the test plan for D4294: smoother formation manouvres.
Sun, Oct 10, 10:34 PM
Freagarach added a comment to rP25953: Add accelerations in unit movement..

When units get pushed, they keep the walking animation although they're idle. See the attached replay (r25958) around ten minutes.

Sun, Oct 10, 9:36 PM
bb requested changes to D4294: smoother formation manouvres.

Thanks for the patch: here is some quick review, to help you improve the patch:

Sun, Oct 10, 9:32 PM
Freagarach committed rP25958: Allow to push items to the front of the ProductionQueue..
Allow to push items to the front of the ProductionQueue.
Sun, Oct 10, 9:07 PM
Freagarach closed D4241: Allow to push items to the front of the ProductionQueue..
Sun, Oct 10, 9:07 PM
Angen added a comment to rP25957: [Petra] Remove global constant from queueplanBuilding.js.

damn, forgot linter

Sun, Oct 10, 9:07 PM
Angen committed rP25957: [Petra] Remove global constant from queueplanBuilding.js.
[Petra] Remove global constant from queueplanBuilding.js
Sun, Oct 10, 9:02 PM
Angen closed D4301: [Petra] Remove global constant from queueplanBuilding.js.
Sun, Oct 10, 9:02 PM
Freagarach committed rP25956: Add two missing hotkeys to the intro.txt..
Add two missing hotkeys to the intro.txt.
Sun, Oct 10, 8:40 PM
Freagarach closed D4284: Add two missing hotkeys to the intro.txt..
Sun, Oct 10, 8:40 PM
Freagarach accepted D4301: [Petra] Remove global constant from queueplanBuilding.js.

Correct, having these kind of globals around is bad.
It is not used anywhere else (which would have been really bad).

Sun, Oct 10, 8:36 PM
Freagarach added a comment to D3544: Regroup members at most once when formation and its members get idle.

One could tell the formation controller that we have have left/resumed the IDLE state and that could trigger the reforming?

Sun, Oct 10, 8:15 PM
Langbart added a comment to D3544: Regroup members at most once when formation and its members get idle.

There is only a very short moment when the idle number on the minimap still "twitches". This is definitely an improvement.

Sun, Oct 10, 3:00 PM
Langbart added a comment to D3544: Regroup members at most once when formation and its members get idle.

With D4217, the number of units is now displayed on the minimap, idle units are alternating the isIdle state from true to false, which causes the number of idle units to change constantly. When I apply this patch to my working copy rP25953, the problem described in ticket #6279 no longer occurs. It fixes it.

Sun, Oct 10, 2:51 PM
Angen added a comment to D4299: Changing a single word in the Introductory Tutorial.

Thank you for the patch

Sun, Oct 10, 2:50 PM
Angen committed rP25955: Fix mention of 'farm' instead of 'field' in tutorial.
Fix mention of 'farm' instead of 'field' in tutorial
Sun, Oct 10, 2:49 PM
Angen closed D4299: Changing a single word in the Introductory Tutorial.
Sun, Oct 10, 2:49 PM
Vulcan added a comment to D4301: [Petra] Remove global constant from queueplanBuilding.js.

Build failure - The Moirai have given mortals hearts that can endure.

Sun, Oct 10, 2:04 PM
Angen published D4301: [Petra] Remove global constant from queueplanBuilding.js for review.
Sun, Oct 10, 2:03 PM
Angen requested changes to D4056: [gameplay] Mercenary camp tooltip correction .

It allows to recruit also another units than mercenaries.
It should be investigated what is actually goal here.
Because now it looks like regular barracks with additional merc units for specific civ.
So tooltip is wrong anyway.

Sun, Oct 10, 1:55 PM
Angen accepted rP25927: Fix black buttons as observer on a GAIA market..
Sun, Oct 10, 11:48 AM
Angen committed rP25954: Fix missing sell buttons.
Fix missing sell buttons
Sun, Oct 10, 11:33 AM
Angen closed D4298: Fix missing sell buttons.
Sun, Oct 10, 11:33 AM
Langbart added a comment to D4297: Fixing a problem where the AI attacked bridges.

Instead of removing the templates, one could create a template_bridge with no health/resistance and let the two used templates inherit from that. But since the templates are wonky and don't add anything, deletion seems like the correct choice.

Sun, Oct 10, 9:26 AM
Freagarach awarded rP25953: Add accelerations in unit movement. a Love token.
Sun, Oct 10, 8:55 AM
Freagarach added inline comments to rP25691: Map flares.
Sun, Oct 10, 8:53 AM
Langbart published D4299: Changing a single word in the Introductory Tutorial for review.
Sun, Oct 10, 8:43 AM
Freagarach accepted D4297: Fixing a problem where the AI attacked bridges.

The reason the bridge had health is because we couldn't have healthless entities earlier (refs. D341, D1268).

Sun, Oct 10, 8:08 AM
Langbart updated the summary of D4296: Layout changes to the replay menu.
Sun, Oct 10, 7:46 AM
Langbart added a comment to D4297: Fixing a problem where the AI attacked bridges.
In D4297#183000, @bb wrote:

If we just want the AI to stop attacking them, I suppose we could also remove the health component and be fine.

If I remove the Health tag it throws errors at me, is there another way?

Sun, Oct 10, 7:34 AM
playstore updated playstore.
Sun, Oct 10, 5:32 AM

Sat, Oct 9

bb committed rP25953: Add accelerations in unit movement..
Add accelerations in unit movement.
Sat, Oct 9, 11:32 PM
bb closed D3200: Stop dodging arrows by spamclicking or patrol: Combine rotations with acceleration..
Sat, Oct 9, 11:32 PM
bb accepted D4297: Fixing a problem where the AI attacked bridges.
Sat, Oct 9, 10:42 PM
bb published D4297: Fixing a problem where the AI attacked bridges for review.

The question in need of answering here is: do bridges need to be entities or are actors enough?

Sat, Oct 9, 10:42 PM
Angen raised a concern with rP25691: Map flares.
Sat, Oct 9, 10:29 PM
bb updated the test plan for D4240: Remove useless persistMatchSettings and handle some which weren't handled.
Sat, Oct 9, 10:18 PM
Vulcan added a comment to D4240: Remove useless persistMatchSettings and handle some which weren't handled.

Build failure - The Moirai have given mortals hearts that can endure.

Sat, Oct 9, 10:17 PM
bb updated the diff for D4240: Remove useless persistMatchSettings and handle some which weren't handled.

A deepfreeze

Sat, Oct 9, 10:16 PM
Angen raised a concern with rP25927: Fix black buttons as observer on a GAIA market..
Sat, Oct 9, 7:49 PM
Langbart updated the summary of D4296: Layout changes to the replay menu.
Sat, Oct 9, 5:09 PM
Vulcan added a comment to D4296: Layout changes to the replay menu.

Build failure - The Moirai have given mortals hearts that can endure.

Sat, Oct 9, 5:09 PM
Langbart updated the diff for D4296: Layout changes to the replay menu.

Changing the buttons from ModernButtonRed back to their original StoneButton style.

Sat, Oct 9, 5:08 PM
Langbart added a comment to D4296: Layout changes to the replay menu.

It seems that in general, the red buttons are used for pupups, but either way, we should make them not less consistent, but more consistent.

Sat, Oct 9, 5:02 PM
Langbart added a comment to D4292: Moving the mininmal resolution to 1280x800.

Small discussion on IRC 0ad-dev (4/Oct/21), increasing width to 1280 meets little resistance, increasing width to 800 meets some opposition who would rather see 720 or 768. Below 768 we would have even less vertical space e.g. for displaying control groups, I would be against it and would take either 800 or 768.

  • 1280x768 or 1280x800 seem to be suitable candidates.

The most important part is the increased width, which would allow a larger minimap, and increasing the width of the summary screen columns, which are too small for some translations (see #6025 - Summary page problem with long words (15/Feb/21)).

Sat, Oct 9, 4:59 PM
Vulcan added a comment to D4298: Fix missing sell buttons.

Build failure - The Moirai have given mortals hearts that can endure.

Sat, Oct 9, 3:53 PM
Angen published D4298: Fix missing sell buttons for review.
Sat, Oct 9, 3:52 PM
carswreckersau updated carswreckersau.
Sat, Oct 9, 11:29 AM
matebiz updated matebiz.
Sat, Oct 9, 10:01 AM
matebiz updated matebiz.
Sat, Oct 9, 10:01 AM
Vulcan added a comment to D4241: Allow to push items to the front of the ProductionQueue..

Build failure - The Moirai have given mortals hearts that can endure.

Sat, Oct 9, 7:49 AM
Freagarach updated the diff for D4241: Allow to push items to the front of the ProductionQueue..

Add semicolon. Fix icon being behind the progressslider.

Sat, Oct 9, 7:49 AM
Freagarach updated the summary of D4241: Allow to push items to the front of the ProductionQueue..
Sat, Oct 9, 7:40 AM
Freagarach edited reviewers for D4290: Adjusting the position of the generic/specific name if only one is displayed, added: bb; removed: Freagarach.

(It looks good in-game though.)

Sat, Oct 9, 7:32 AM
Freagarach published D4292: Moving the mininmal resolution to 1280x800 for review.
Sat, Oct 9, 7:02 AM

Fri, Oct 8

Vulcan added a comment to D3037: Add gui scale to options.

Build failure - The Moirai have given mortals hearts that can endure.

Fri, Oct 8, 9:30 PM
Angen updated the diff for D3037: Add gui scale to options.

rebase and some cleaning

Fri, Oct 8, 9:29 PM
techservices updated techservices.
Fri, Oct 8, 9:44 AM
techservices updated techservices.
Fri, Oct 8, 9:44 AM
autobuild committed rP25952: [i18n] Updated POT and PO files..
[i18n] Updated POT and PO files.
Fri, Oct 8, 9:15 AM
Freagarach added a comment to D4296: Layout changes to the replay menu.

It seems that in general, the red buttons are used for pupups, but either way, we should make them not less consistent, but more consistent.

Fri, Oct 8, 8:24 AM

Thu, Oct 7

Freagarach added a comment to D4204: Let PetraAI provide the resource map config instead of hardcoding it in the sharedScript..

I guess the same as the territory and border maps, which are also Petra specific, not player specific? So maybe some Petra-shared map?

Thu, Oct 7, 6:53 PM
Freagarach published D4296: Layout changes to the replay menu for review.
Thu, Oct 7, 4:18 PM
Vulcan added a comment to D4290: Adjusting the position of the generic/specific name if only one is displayed.

Build failure - The Moirai have given mortals hearts that can endure.

Thu, Oct 7, 3:39 PM
Langbart updated the diff for D4290: Adjusting the position of the generic/specific name if only one is displayed.

Trying to solve with getTextSize().height.

Thu, Oct 7, 3:39 PM