Page MenuHomeWildfire Games
Feed All Stories

Today

Angen added inline comments to D2360: Fix entities not retaining repair order after target transforms..
Wed, Jan 22, 8:57 AM
Stan added a comment to D2528: Performance improvements to VertexBuffer.

Some notes. Can you add yourself to the credits? Keep up the great work!

Wed, Jan 22, 8:09 AM
dm updated the diff for D2528: Performance improvements to VertexBuffer.

With more testing, in high number deathmatch, there was still high usage in CVertextBuffer::Release (gprof)

Wed, Jan 22, 2:42 AM

Yesterday

Nescio added a comment to D2578: update structure template tooltips.

The structures/maur_defense_tower is missing a ..

Not done: this patch standardizes <Identity> tooltips. <Upgrade> tooltips could be done in a future patch.

Just wondering: train seems like D1245 and recruit more like what is done when clicking the button? I think this can be comitted as is and we can discuss "train" vs "recruit" elsewhere, if you want?

https://trac.wildfiregames.com/wiki/EnglishStyleGuide#Actions says “train (human units)” and that's also the verb used in the aura descriptions and technology tooltips (“training time”), so for consistency's sake we should stick with that in this patch.
A future patch could replace train with conscript/enlist/recruit, if people think that's clearer.

Tue, Jan 21, 9:33 PM
Vulcan added a comment to D2578: update structure template tooltips.

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

Tue, Jan 21, 9:27 PM
Nescio updated the diff for D2578: update structure template tooltips.

brit and gaul wall tower

Tue, Jan 21, 9:23 PM
Vulcan added a comment to D2591: Javelinist → Javelineer.

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

Tue, Jan 21, 9:17 PM
Nescio created D2591: Javelinist → Javelineer.
Tue, Jan 21, 9:13 PM
Freagarach requested changes to D2578: update structure template tooltips.

Just wondering: train seems like D1245 and recruit more like what is done when clicking the button? I think this can be comitted as is and we can discuss "train" vs "recruit" elsewhere, if you want?

Tue, Jan 21, 9:11 PM
ValihrAnt added a comment to D2585: Cavalry archer speed adjustment.

Then why not lower heavy skirmisher speed?

Fewer to test, basically. An additional benefit of the change being more minor, which is something I aimed to do when balancing. Avoid making big sweeping changes and if a change turns out to be too minuscule it can be further adjusted in the future.

Or raise cavalry javelinist speed?

Obviously, this would make javelin cavalry stronger, which I think is very unneeded as they're in a good spot balance wise currently.

By the way, this makes elephant archers slower, since they have the horse archer as their parent. Also, keep in mind there are two cavalry speed technologies (+10% each), from which elephantry and infantry don't benefit.

Didn't actually realize that was the case, will fix it in the patch soon. Getting the first speed upgrade will actually restore cav archer speed back to slightly higher than initially.

Tue, Jan 21, 9:05 PM
Vulcan added a comment to D2360: Fix entities not retaining repair order after target transforms..

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

Tue, Jan 21, 8:42 PM
Freagarach updated the diff for D2360: Fix entities not retaining repair order after target transforms..
  • Remove if in addBuilder.
  • AddSingleBuilder -> AddBuilderHelper.
  • Don't say orders have changed when they might have not been.
Tue, Jan 21, 8:38 PM
Freagarach added inline comments to D2360: Fix entities not retaining repair order after target transforms..
Tue, Jan 21, 8:38 PM
Vulcan added a comment to D2328: Unify GetRange calls in UnitAI..

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

Tue, Jan 21, 8:24 PM
Freagarach updated the diff for D2328: Unify GetRange calls in UnitAI..

Added undefined type.

Tue, Jan 21, 8:20 PM
bb accepted rP23392: Landscape and Daytime gamesetup options for the Unknown, Danubius and Polar Sea….

bug disappeared

Tue, Jan 21, 4:55 PM
bb accepted rP23374: Gamesetup class rewrite, fixes #5322, refs #5387..

Reported bug has been fixed, one can notice that the animation changed in the case when switching from 2 to 1 tab (there now is a gap between the panelButtons and the settingsPanel). New version fine with me too

Tue, Jan 21, 4:54 PM
Stan added a comment to D2590: delete some unused textures.

That sounds nice.

Tue, Jan 21, 2:39 PM
Nescio added a comment to D2590: delete some unused textures.

I guess committing this means adding a PR to delenda est :D

If you make a https://github.com/0ADMods/deprecated_art mod I could these files to there, and Delenda Est could then be fixed by simply adding a dependency.

Tue, Jan 21, 2:30 PM
Nescio added a comment to D2502: adjust wall footprints.

This patch will also solve gaps like this:

Tue, Jan 21, 2:22 PM
Stan added a comment to D2590: delete some unused textures.

I guess committing this means adding a PR to delenda est :D

Tue, Jan 21, 2:17 PM
Stan added a comment to D2584: Disable QuickLoad during multiplayer.

I don't know what you refer to when you speak about MP savegame features, because so far there is only one MP savegame feature on the table, which is picking a savegame, starting a host, and then running that match.

Tue, Jan 21, 2:16 PM
Nescio added a comment to D2590: delete some unused textures.

Well, I don't know either, so I'll leave it up to you to figure out what should be done with deprecated art.

Tue, Jan 21, 2:14 PM
elexis added a comment to D2584: Disable QuickLoad during multiplayer.
In D2584#108308, @Stan wrote:

I think it would be nicer to have the MP savegame features

Tue, Jan 21, 2:07 PM
Stan added a comment to D2590: delete some unused textures.

Maybe those textures should be moved to the old public folder in https://trac.wildfiregames.com/browser/art_source/trunk/old_public

Tue, Jan 21, 1:58 PM
Vulcan added a comment to D2590: delete some unused textures.

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

Tue, Jan 21, 1:50 PM
elexis updated the summary of D2584: Disable QuickLoad during multiplayer.
Tue, Jan 21, 1:49 PM
Nescio created D2590: delete some unused textures.
Tue, Jan 21, 1:46 PM
Nescio updated the test plan for D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:39 AM
Vulcan added a comment to D2589: correct watermark aspect ratio.

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

Tue, Jan 21, 11:34 AM
Nescio updated the test plan for D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:32 AM
Vulcan added a comment to D2589: correct watermark aspect ratio.

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

Tue, Jan 21, 11:31 AM
Nescio updated the test plan for D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:28 AM
Nescio updated the summary of D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:28 AM
Nescio updated the test plan for D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:27 AM
Vulcan added a comment to D2589: correct watermark aspect ratio.

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

Tue, Jan 21, 11:27 AM
Nescio updated the diff for D2589: correct watermark aspect ratio.

192 width

Tue, Jan 21, 11:26 AM
Nescio updated the diff for D2589: correct watermark aspect ratio.

256 width

Tue, Jan 21, 11:25 AM
Nescio created D2589: correct watermark aspect ratio.
Tue, Jan 21, 11:22 AM
vladislavbelov added a comment to D2531: Analysis of unique_range assertion failure on MacOS.
In D2531#108430, @Stan wrote:

I've reproduced that assertion locally on macOS 10.14.6 in debug-mode. I've figured out the reason: it seems that OSAtomicAdd64 doesn't work as expected in debug mode, maybe address was misaligned.

Can it be fixed?

Tue, Jan 21, 11:17 AM · Contributors
Nescio edited reviewers for D2493: [gameplay] make siege engines uncapturable, added: ValihrAnt; removed: borg-.
Tue, Jan 21, 10:55 AM
Vulcan added a comment to D2568: gui: remove empty space between logo and buttons.

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

Tue, Jan 21, 10:48 AM
Nescio updated the summary of D2568: gui: remove empty space between logo and buttons.
Tue, Jan 21, 10:45 AM
Nescio updated the test plan for D2568: gui: remove empty space between logo and buttons.
Tue, Jan 21, 10:43 AM
Nescio updated the diff for D2568: gui: remove empty space between logo and buttons.

C = 50%, A, B, D, E, F 10%.

Tue, Jan 21, 10:41 AM
Nescio accepted rP23422: Delete duplicate and deprecated unit icons..
Tue, Jan 21, 10:17 AM
Stan added a comment to D2531: Analysis of unique_range assertion failure on MacOS.

I've reproduced that assertion locally on macOS 10.14.6 in debug-mode. I've figured out the reason: it seems that OSAtomicAdd64 doesn't work as expected in debug mode, maybe address was misaligned.

Tue, Jan 21, 10:02 AM · Contributors
vladislavbelov added a comment to D2531: Analysis of unique_range assertion failure on MacOS.

I've reproduced that assertion locally on macOS 10.14.6 in debug-mode. I've figured out the reason: it seems that OSAtomicAdd64 doesn't work as expected in debug mode, maybe address was misaligned.

Tue, Jan 21, 9:25 AM · Contributors
Stan requested verification of rP23422: Delete duplicate and deprecated unit icons..
Tue, Jan 21, 8:13 AM

Mon, Jan 20

Stan committed rP23428: Fix two incorrect icons..
Fix two incorrect icons.
Mon, Jan 20, 11:58 PM
Stan committed rP23427: Use a non transparent material to improve performance..
Use a non transparent material to improve performance.
Mon, Jan 20, 11:13 PM
vladislavbelov added a comment to rP23426: Removes unused gfx::GetMonitorSize function.

Refs D2476.

Mon, Jan 20, 10:56 PM
vladislavbelov committed rP23426: Removes unused gfx::GetMonitorSize function.
Removes unused gfx::GetMonitorSize function
Mon, Jan 20, 10:54 PM
vladislavbelov added a comment to rP23425: Removes unused MemorySizeMiB. Refs D2476..

MemorySizeMiB and its usage was added in rP9546, was removed in rP13892.

Mon, Jan 20, 10:43 PM
vladislavbelov committed rP23425: Removes unused MemorySizeMiB. Refs D2476..
Removes unused MemorySizeMiB. Refs D2476.
Mon, Jan 20, 10:24 PM
vladislavbelov committed rP23424: Saves sidebar/bottombar width/height between Atlas sessions.
Saves sidebar/bottombar width/height between Atlas sessions
Mon, Jan 20, 9:58 PM
vladislavbelov closed D2583: Saves sidebar/bottombar width/height between Atlas sessions.
Mon, Jan 20, 9:58 PM
vladislavbelov added a comment to rP23423: Adds non-strict search to the Atlas entity list.

This is a great improvement, many thanks!

Mon, Jan 20, 9:58 PM
Nescio added a comment to rP23423: Adds non-strict search to the Atlas entity list.

This is a great improvement, many thanks!

Mon, Jan 20, 9:54 PM
vladislavbelov committed rP23423: Adds non-strict search to the Atlas entity list.
Adds non-strict search to the Atlas entity list
Mon, Jan 20, 9:43 PM
vladislavbelov closed D2542: Adds smart search to the Atlas entity list.
Mon, Jan 20, 9:43 PM
Nescio added a comment to D2554: wall_med → wall_medium.

Given it was committed in rP6904 I assume checkrefs.pl --check-unused reports it?

Yes it does, amongst hundreds of other files.
grep -r decal_struct_wall_med didn't yield any results either.

Changes in collada do not matter :) It's just best to avoid them when there is no reason :)

Blame arcanist; I have no idea what those .dae files are, I didn't even open them.

Mon, Jan 20, 9:42 PM
Nescio added a comment to D2585: Cavalry archer speed adjustment.

Initially, I did 0.90 but then they were slower than Heavy Skirmishers, which didn't feel too correct. When I tried 0.95 (in a single-player test environment) I didn't feel like the change was significant enough. So 0.92 was a nice middle ground.

Then why not lower heavy skirmisher speed? Or raise cavalry javelinist speed? (Again, just curious.)
By the way, this makes elephant archers slower, since they have the horse archer as their parent. Also, keep in mind there are two cavalry speed technologies (+10% each), from which elephantry and infantry don't benefit.

Mon, Jan 20, 9:39 PM
ValihrAnt added a comment to D2585: Cavalry archer speed adjustment.

Out of curiosity, why exactly 0.92, not 0.95, 0.90, or 0.85?

Initially, I did 0.90 but then they were slower than Heavy Skirmishers, which didn't feel too correct. When I tried 0.95 (in a single-player test environment) I didn't feel like the change was significant enough. So 0.92 was a nice middle ground.

(I guess it takes a bit of time, but it wouldn't hurt to document the test sample (i.e. approximate number of matches, list of participating players who may be considered either capable of judging or capable of playing representatively for competitive players. If its easier, one could also just upload replays and perhaps a link to the mod. The purpose of the list of test participant would allow (1) the reviewers to determine in how far the patch was tested and (2) later consumers of the patches (a24 players and devs) to find identify which players were capable of judging, or what might have gone wrong or right in case it was a fail or success.)

Changing just speed is my idea. Christmas wanted the changes to include a reduction to speed and range, but an increase to dmg. badosu suggested having a small (0.3 to 0.5) damage increase to make up for the reduced speed. Then lastly I have Boudica and Stockfish agreeing with how they are nerfed.
Test match wise I have participated in 8 matches were cavalry archers were used. Though, the matches were spread across the mod versions, which I only have the last one on hand. 2 of the matches had cavalry archers as the main army, in the others they were used for rushing.

Could this make skirmisher cavalry rushes too strong?

I don't see how this makes skirmisher cavalry rushes any stronger. Previously skirm cav struggled to get even 1 hit off on camels running away, now they should be able to fire off at least one shot. Still, when the camels are back at home they will simply be laying suppressive fire from the safety of the CC due to their vastly superior range.

Mon, Jan 20, 9:28 PM
Stan added a comment to D2554: wall_med → wall_medium.

Given it was committed in rP6904 I assume checkrefs.pl --check-unused reports it?

Mon, Jan 20, 9:18 PM
Nescio added a comment to D2554: wall_med → wall_medium.

Also, I didn't rename, art/meshes/props/decal_struct_wall_med.dae, because I'm unsure what to do with it; I couldn't find any file that uses it.

Mon, Jan 20, 8:51 PM
Nescio added a comment to D2554: wall_med → wall_medium.

Any reason for those collada edits? I know Linux needs EOF to be on a newline, but stuff handling DAE seems not to care at all...

Did I break something? arcanist was complaining, first about mime-type (which I set to text/plain), then about something:

>>> Lint for binaries/data/mods/public/art/meshes/structural/brit_wall_medium.dae:
Mon, Jan 20, 8:45 PM
Nescio raised a concern with rP23422: Delete duplicate and deprecated unit icons..

Something went wrong with the gaul_infantry_slinger.png, which should have been this:


and athen_hero_themistocles.png:

Mon, Jan 20, 8:39 PM
Stan added a comment to D2554: wall_med → wall_medium.

If changing ao files, strip the _AO suffix, as they already are in an ao folder (I only do it on a case to case basis)
Any reason for those collada edits? I know Linux needs EOF to be on a newline, but stuff handling DAE seems not to care at all...

Mon, Jan 20, 8:36 PM
Vulcan added a comment to D2554: wall_med → wall_medium.

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

Mon, Jan 20, 8:32 PM
Nescio updated the diff for D2554: wall_med → wall_medium.
Mon, Jan 20, 8:28 PM
Nescio added a reviewer for D2551: rename some structure icons: Stan.
Mon, Jan 20, 8:10 PM
Vulcan added a comment to D2588: delete unused duplicate elephant icon.

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

Mon, Jan 20, 8:04 PM
Stan accepted D2588: delete unused duplicate elephant icon.
Mon, Jan 20, 8:02 PM
Nescio created D2588: delete unused duplicate elephant icon.
Mon, Jan 20, 8:00 PM
Stan committed rP23422: Delete duplicate and deprecated unit icons..
Delete duplicate and deprecated unit icons.
Mon, Jan 20, 7:37 PM
Stan closed D2586: delete duplicate and deprecated unit icons.
Mon, Jan 20, 7:37 PM
Nescio added a comment to D2568: gui: remove empty space between logo and buttons.

Perhaps it's easier if you'd state the percentages how you'd like to distribute the empty space?
A: space between top and logo
B: space between logo and buttons
C: space between buttons and project box
D: space between project box and water mark
E: space between water mark and bottom
(The total vertical space currently required is 660 (100 for the logo, 200 for the buttons, 266 for the project box, and 84 for the watermark), which leaves 60 at 1280×720, 420 at 1920×1080, and 1500 at 3840×2160.)

Mon, Jan 20, 6:53 PM
ValihrAnt created D2587: Loom buff.
Mon, Jan 20, 6:32 PM
Stan accepted D2586: delete duplicate and deprecated unit icons.

Looks good to me :) Thanks for the patch.

Mon, Jan 20, 6:22 PM
elexis added a comment to D2568: gui: remove empty space between logo and buttons.

Perhaps the logo can be moved something like 25-100px depending on resolution, perhaps a percent number to the bottom, and then some margin to the buttons?

Mon, Jan 20, 6:06 PM
Nescio added a comment to D2568: gui: remove empty space between logo and buttons.

Perhaps it would look best if the space between the top of the window and the product logo, between the product logo and the mainmenu buttons, and between the main menu buttons and the top of the project info box, and the bottom of the project info box and the bottom of the window would be divided uniformly?

This is what you meant, right?


I still think sticking with top-down and bottom-up looks better.

Mon, Jan 20, 5:50 PM
elexis added a comment to D2585: Cavalry archer speed adjustment.

Test plan: Changes were tested on the balance mod.

(I guess it takes a bit of time, but it wouldn't hurt to document the test sample (i.e. approximate number of matches, list of participating players who may be considered either capable of judging or capable of playing representatively for competitive players. If its easier, one could also just upload replays and perhaps a link to the mod. The purpose of the list of test participant would allow (1) the reviewers to determine in how far the patch was tested and (2) later consumers of the patches (a24 players and devs) to find identify which players were capable of judging, or what might have gone wrong or right in case it was a fail or success.)

Mon, Jan 20, 5:37 PM
Nescio added a comment to D2586: delete duplicate and deprecated unit icons.

Personally I think icons ought to match the actor. The ptol archer uses the mace actor, which looks the same as the athen actor, so it would make sense for all three to use the same icon.
Both brit and gaul slinger deserve a new icon, I believe their actors were separated and updated last year.

Mon, Jan 20, 5:37 PM
Nescio added a comment to D2568: gui: remove empty space between logo and buttons.

Regardless of the implementation, the productLogo would need to be moved to a different parent object.

Do you mean the 0 A.D. at the top or the Wildfire Games at the bottom?

One could make a mockup with different size numbers and then decide if it looks better or not (for 3840×2160).
Actually another possibility would be to use percent numbers. So perhaps one can use something like 10% +/- offset for the product logo, so that its not aligned at the utter top for large resolutions, only for small ones. (Its quite boring to compute / implement, so I guess one can also argue with the best implementation being too boring to implement and this one being the best one that is feasible given motivation.)

(The idea was actually to improve things for 1280×720; if it works for a small screen, it typically works for a higher resolution as well; the reverse is not true; anyway, I'll do some tinkering with the numbers and see what I can get.)

Mon, Jan 20, 5:33 PM
Stan added a comment to D2586: delete duplicate and deprecated unit icons.

Keep in mind many ship, siege, and support units currently share a single icon for templates of different civs.

Mon, Jan 20, 5:27 PM
elexis added a comment to D2568: gui: remove empty space between logo and buttons.

This is the patch I uploaded at https://code.wildfiregames.com/rP23387#40853 so it would be correct to credit that.

did you mean you want your username explicitly mentioned?

Yes. (I don't care about this one line changed here, just in general. Discussion != patch)

Mon, Jan 20, 5:26 PM
Nescio added a comment to D2586: delete duplicate and deprecated unit icons.

Keep in mind many ship, siege, and support units currently share a single icon for templates of different civs.

Mon, Jan 20, 5:24 PM
Nescio added a comment to D2251: AI: replace Cavalry with new FastMoving class.

@elexis, you make valid points. Nonetheless, I still believe this patch in its current form is an improvement over the status quo.

Mon, Jan 20, 5:21 PM
Stan added a comment to D2586: delete duplicate and deprecated unit icons.

For the ones using an incorrect civ, wouldn't it make more sense to duplicate the icon to have both brit and celt, mace and athen etc?

Maybe, though that would mean having two exactly identical images. Wasn't it your intention to try reducing the download size?

Mon, Jan 20, 5:21 PM
Nescio added inline comments to D2586: delete duplicate and deprecated unit icons.
Mon, Jan 20, 5:18 PM
Nescio updated the summary of D2586: delete duplicate and deprecated unit icons.
Mon, Jan 20, 5:17 PM
Nescio added a comment to D2586: delete duplicate and deprecated unit icons.

For the ones using an incorrect civ, wouldn't it make more sense to duplicate the icon to have both brit and celt, mace and athen etc?

Maybe, though that would mean having two exactly identical images. Wasn't it your intention to try reducing the download size?

Mon, Jan 20, 5:15 PM
Nescio added a comment to D2568: gui: remove empty space between logo and buttons.

This is the patch I uploaded at https://code.wildfiregames.com/rP23387#40853 so it would be correct to credit that.

Again, I'm not sure what you mean, the summary already starts with “As discussed in rP23387”. Or did you mean you want your username explicitly mentioned? (No problem!)

Perhaps it would look best if the space between the top of the window and the product logo, between the product logo and the mainmenu buttons, and between the main menu buttons and the top of the project info box, and the bottom of the project info box and the bottom of the window would be divided uniformly?

I suppose I could try out how that would look, although I don't think floating elements would be an improvement.
Is there a XML equivalent to (La)TeX \vfill?

Mon, Jan 20, 5:11 PM
Stan added a comment to D2586: delete duplicate and deprecated unit icons.

For the ones using an incorrect civ, wouldn't it make more sense to duplicate the icon to have both brit and celt, mace and athen etc?

Mon, Jan 20, 5:09 PM
Nescio added a comment to D2583: Saves sidebar/bottombar width/height between Atlas sessions.

Thanks, I've tried it (on Fedora 31 with Gnome desktop), and it works as described in the summary and test plan.

Mon, Jan 20, 4:56 PM
Nescio added a comment to D2585: Cavalry archer speed adjustment.

Out of curiosity, why exactly 0.92, not 0.95, 0.90, or 0.85?

Mon, Jan 20, 4:54 PM
Vulcan added a comment to D2586: delete duplicate and deprecated unit icons.

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

Mon, Jan 20, 4:54 PM
Nescio added inline comments to D2586: delete duplicate and deprecated unit icons.
Mon, Jan 20, 4:51 PM
Nescio created D2586: delete duplicate and deprecated unit icons.
Mon, Jan 20, 4:48 PM