Page MenuHomeWildfire Games

nani
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 18 2018, 10:57 PM (153 w, 28 m)

Recent Activity

Jun 22 2021

nani added inline comments to D3243: Gamesetup cleanup - Separate Game Settings from the GUI Gamesetup.
Jun 22 2021, 9:18 PM
nani awarded D3243: Gamesetup cleanup - Separate Game Settings from the GUI Gamesetup a Like token.
Jun 22 2021, 9:17 PM

Jun 21 2021

nani accepted rP25381: Enable production entities to autoqueue..

But I guess it is not enough, since @nani did not resign from this commit yet.

Jun 21 2021, 1:57 PM

Jun 20 2021

nani added a comment to rP25381: Enable production entities to autoqueue..

@nani has the autoqueue been adapted enough to your liking?

Tried it, I think it behaves as it should now. As I said before, we could add some negative characteristics like a % extra time to train an unit or something similar but that can be added with ease in the next alpha if in this one we detect the function is too overpowered (personally I think is not, but I like it because I have to click less and thus don't get my wrist all work up, see https://en.wikipedia.org/wiki/Carpal_tunnel_syndrome )

Jun 20 2021, 10:29 PM

Jun 19 2021

nani added a comment to D3687: Color incompatible mods and add filter.
In D3687#166315, @Stan wrote:

Not sure what I did with ffm_visiblity but I got this

ERROR: JavaScript error: gui/gamesetup/NetMessages/GameRegisterStanza~badmod.js line 1 GameRegisterStanza is not defined @gui/gamesetup/NetMessages/GameRegisterStanza~badmod.js:1:1 onPress@gui/pregame/MainMenuItems.js:90:13 performButtonAction@gui/pregame/MainMenuItemHandler.js:77:9 pressButton@gui/pregame/MainMenuItemHandler.js:63:10

A commit changed the name of the class.

Jun 19 2021, 2:13 AM
nani added a comment to D3719: GameSetup GUI cleanup - renaming & moving stuff around..

Why are the files names different from the class names? Makes search twice difficult.

Jun 19 2021, 2:12 AM

Jun 13 2021

nani awarded D2936: Allow limiting the max number of corpses simultaneously visible in the game a Evil Spooky Haunted Tree token.
Jun 13 2021, 5:45 PM

Jun 8 2021

nani added a comment to rP25381: Enable production entities to autoqueue..
In rP25381#52929, @nani wrote:

Well, that makes no sense

As written, it does, it makes auto queue slightly worse than manual queueing.

The previous problem makes it worse when you want to delete the unit that is in training (or all the units in the queue) and the auto queue starts to add more units to queue before you can delete them.

That is more problematic, we probably de-activate the auto queue if the production queue is manually changed.

Creating ticket.

Jun 8 2021, 6:51 PM
nani added a comment to rP25381: Enable production entities to autoqueue..
In rP25381#52928, @Stan wrote:

@nani Apparently it's intended so it's no OP.

Jun 8 2021, 6:18 PM
nani raised a concern with rP25381: Enable production entities to autoqueue..

Although the auto-queue works it has some pretty apparent broken functionality:

Jun 8 2021, 6:14 PM

Jun 7 2021

nani awarded D4108: Adds FreeType support to the engine a Love token.
Jun 7 2021, 6:52 PM
nani awarded D4108: Adds FreeType support to the engine a Love token.
Jun 7 2021, 6:52 PM

May 27 2021

nani awarded D4021: Harder Better Faster Tileclasses a Like token.
May 27 2021, 1:12 AM

May 24 2021

nani added a comment to D4009: [WIP] - Adding some buttons to the lobby page.
In D4009#172642, @nani wrote:

Maybe also a "last summary" button would be nice as is domething people always wan to check after the game and comment in the lobby

I can pass you the code for it here if willing

I added a 5th button, you can add the code to it. Would be great.

May 24 2021, 11:04 PM

May 23 2021

nani added a reviewer for D4009: [WIP] - Adding some buttons to the lobby page: ffffffff.
May 23 2021, 1:12 PM
nani removed a reviewer for D4009: [WIP] - Adding some buttons to the lobby page: Restricted Owners Package.
In D4009#172645, @Angen wrote:

isnt this too much buttons ?

May 23 2021, 1:11 PM
nani added a comment to D4009: [WIP] - Adding some buttons to the lobby page.

Maybe also a "last summary" button would be nice as is domething people always wan to check after the game and comment in the lobby

May 23 2021, 10:57 AM
nani awarded D4009: [WIP] - Adding some buttons to the lobby page a Love token.
May 23 2021, 10:57 AM

May 22 2021

nani awarded D4003: Removes unnecessary z-level from GUI rendering a Like token.
May 22 2021, 8:49 PM
nani added a comment to D4003: Removes unnecessary z-level from GUI rendering.
In D4003#172443, @nani wrote:

Doesn't the summary screen use z-index for some icons pretty extensibly? the same for the tech tree, the barter dialog and the teams dialog in session.

z-level != z-index, the latter one is used for the mentioned objects.

May 22 2021, 6:39 PM
nani added a comment to D4003: Removes unnecessary z-level from GUI rendering.

Doesn't the summary screen use z-index for some icons pretty extensibly? the same for the tech tree, the barter dialog and the teams dialog in session.

May 22 2021, 5:37 PM
nani added a comment to D4003: Removes unnecessary z-level from GUI rendering.

I guess this issue is old:

Yes, it's pretty old. Since we don't know how to draw text there properly. Because "borders" are not real borders but a single background texture. And for a single texture you need to crop the text, but there is no information about the "border" width. So the simplest solution is to split the single texture into background and borders to draw them after the text.

May 22 2021, 5:35 PM

May 20 2021

nani added a comment to D3985: Split graphics options in two..

Unfortunately, proper scrolling in the GUI is not possible for now because the GUI renderer won't support it. It needs more fundamental changes before that becomes possible.

May 20 2021, 11:26 PM

May 19 2021

nani awarded D3532: Pick map in map browser upon hitting return. a Pterodactyl token.
May 19 2021, 1:48 PM
nani added a comment to D3532: Pick map in map browser upon hitting return..

I find myself agreeing with @nani - this would be convenient.
However, what I would do is that 'Enter' when the input box is in Focus selects the first grid item, and then 'Enter' again actually submits the selection (so you can also navigate - which may or may not work itself, don't recall).

@Freagarach: I can try and take over for you, I'm more familiar with this code than you are.

May 19 2021, 1:46 PM

May 17 2021

nani added a comment to D3968: Allow mods to declare they are 'checksum compatible', i.e. don't change checksums..

As for if we should check what the mods changes, I think @wraitii is right that we should just let the user and creator decide for themselves.
Trying to analize if the mod is OOS safe is IMHO a waste of time as there a many way to mod and it would be a frustration more than anything else.
Maybe there could be a notice for mods with "compatibleMods" true or the engine could analyze what folders the mods touches but I think with the notice alone should be enough.
Also a notice if the host or someone else is using an extra mod would be a good idea.

May 17 2021, 9:20 PM
nani awarded D3972: Implements cascade shadows a Like token.
May 17 2021, 1:15 AM

May 14 2021

ffffffff awarded D3957: Make fruit (berries) obstruction smaller a Like token.
May 14 2021, 3:29 PM

May 13 2021

nani added a comment to D3957: Make fruit (berries) obstruction smaller.

Is intended for those who don't overwrite the parent setting, which means is intended.

May 13 2021, 5:16 PM
nani requested review of D3957: Make fruit (berries) obstruction smaller.
May 13 2021, 4:59 PM

May 4 2021

nani awarded D3833: Map browser: input filter, also filter random map entry a Like token.
May 4 2021, 9:08 PM

Apr 24 2021

nani awarded D3865: Auto-Queue feature a Like token.
Apr 24 2021, 6:14 AM · Contributors
nani added a comment to D3865: Auto-Queue feature.

You might want to add some hotkeys for this.
I suggest these ones I already use that I think work well:
Alt+q to enable auto queue
Alt+w to disable auto queue

Apr 24 2021, 6:13 AM · Contributors

Apr 21 2021

nani added a comment to D3532: Pick map in map browser upon hitting return..

@Freagarach this D3833: Map browser: input filter, also filter random map entry

Apr 21 2021, 3:27 AM

Apr 11 2021

nani updated the Trac tickets for D3833: Map browser: input filter, also filter random map entry.
Apr 11 2021, 12:23 AM
nani requested review of D3833: Map browser: input filter, also filter random map entry.
Apr 11 2021, 12:13 AM

Apr 6 2021

nani added a comment to P249 Code to profile all javascript simulation components and helper classes methods. Need to be loaded after all the other files..

(I add this code to the folder /simulation/components and /simulation/helpers/ )

Apr 6 2021, 9:26 PM
nani updated subscribers of P249 Code to profile all javascript simulation components and helper classes methods. Need to be loaded after all the other files..

@wraitii testing this on a casual multiplayer game gives me that the most expensive simulation update looks to be on the component timer.update and all the handlers is has to process , maybe this will help find a way to finally find why javascript has this terrible simulation performance

Apr 6 2021, 9:22 PM

Apr 4 2021

nani changed the visibility for P249 Code to profile all javascript simulation components and helper classes methods. Need to be loaded after all the other files..
Apr 4 2021, 9:25 PM
nani created P249 Code to profile all javascript simulation components and helper classes methods. Need to be loaded after all the other files..
Apr 4 2021, 9:24 PM
nani awarded D3787: Implement an actor quality setting, with option. a Love token.
Apr 4 2021, 2:51 AM

Apr 2 2021

nani added a comment to D3603: Water-Patch.

@DanW58 what happened? This diff looked so promising would be a pity to be left to accumulate dust.

Apr 2 2021, 1:14 AM
nani awarded D3603: Water-Patch a Love token.
Apr 2 2021, 1:12 AM

Mar 31 2021

nani accepted D3766: Further hotkey fixes: don't retrigger on key release..

At first glance seem to work as advertised, testing different hotkeys doesn't produce "retriggers" anymore. Haven't look at the code. Functional part of the diff: accepted.

Mar 31 2021, 1:57 PM

Mar 29 2021

nani added a comment to D3767: Show non-default hotkeys that appear in JSON spec files in the hotkey editor..
In D3767#163831, @nani wrote:

What if the user defines his own custom hotkeys in user.cfg which you didn't specify in GUI/hotkeys/spec/**.json. I know I'm stretching the limits but will they be deleted?

Only if they're empty/unmapped.
It seems to me you're trying to get at some feature of your mod to automatically create hotkeys or something like that? One option would be to write a JSON file with the hotkeys from your mod code (it'll get written to the user mod, but that's fine).

If you actually need this feature, I could also make it more configurable that empty hotkeys get deleted.

Mar 29 2021, 7:57 PM
nani added a comment to D3767: Show non-default hotkeys that appear in JSON spec files in the hotkey editor..

What if the user defines his own custom hotkeys in user.cfg which you didn't specify in GUI/hotkeys/spec/**.json. I know I'm stretching the limits but will they be deleted?

Mar 29 2021, 7:28 PM
nani added a comment to D3767: Show non-default hotkeys that appear in JSON spec files in the hotkey editor..

I'm even more confused now, where are things stored, how are they handled, there seems to be at least 3 independent places where hotkey config is being modified and stored.

Mar 29 2021, 7:15 PM
nani added a comment to D3766: Further hotkey fixes: don't retrigger on key release..

Should 'Down' still be sent? This is the 'repeating' event.

Mar 29 2021, 7:08 PM
nani added inline comments to D3767: Show non-default hotkeys that appear in JSON spec files in the hotkey editor..
Mar 29 2021, 7:01 PM
nani added inline comments to D3641: Improve & translate hotkey name/descriptions.
Mar 29 2021, 6:02 PM

Mar 26 2021

nani accepted rP23701: Implement keyDown event .
In rP23701#50076, @bb wrote:

That is a feature not a bug. Keydown should repeat, keyPress should fire just once. StopUnits has been explicitly discussed to be keyDown (hence repeating), (One reason being ungarrison, another being units going the attack)

Mar 26 2021, 1:11 PM
nani raised a concern with rP23701: Implement keyDown event .

KeyDown event still keeps being called after being pressed down.
Example: add a warn inside stopUnits functions and see how it repeats.

Mar 26 2021, 12:59 PM

Mar 25 2021

nani added a comment to D3737: MP netcode: allow observers to lag however much they want..

Good option to add nonetheless.

Mar 25 2021, 3:31 PM
nani added a comment to D3737: MP netcode: allow observers to lag however much they want..
In D3737#162860, @nani wrote:

Given this is not a technical issue but gameplay decision maybe would be better do a poll in forums?

I'll just add a config option that defaults to "wait for observers" for now.

Mar 25 2021, 3:20 PM
nani added a comment to D3737: MP netcode: allow observers to lag however much they want..

Given this is not a technical issue but gameplay decision maybe would be better do a poll in forums?

Mar 25 2021, 2:47 PM

Mar 24 2021

nani added a comment to D3737: MP netcode: allow observers to lag however much they want..

What if the host is an observer? Host should then always be on sync.
I am aginst this as a default, it will break the experience of observing games online and make everyone confused.

Mar 24 2021, 10:54 PM

Mar 18 2021

nani awarded D3243: Gamesetup cleanup - Separate Game Settings from the GUI Gamesetup a Baby Tequila token.
Mar 18 2021, 9:54 AM
nani added a comment to D3243: Gamesetup cleanup - Separate Game Settings from the GUI Gamesetup.

As is it now, this looks like a frankentrain mixed with the two old systems that still linger in the gamesetup code.
I suggest to try it on an simpler case instead of going for the last boss, for example the options page looks like an easy candidate to change and fom there we can feel how it would translate to other more complex pages like gamesetup once we have the idea flesh out.

Mar 18 2021, 9:54 AM

Feb 27 2021

nani added a comment to D2936: Allow limiting the max number of corpses simultaneously visible in the game.

I made a new battle profile map and profiled it.


Vanilla

Limited to 100 corpses

Limited to 50 corpses


(D3554 is needed)

Feb 27 2021, 7:14 PM
nani added a comment to D2936: Allow limiting the max number of corpses simultaneously visible in the game.

You need an ordered set by insertion, all the others options are just doing the same with extra steps. Maybe you can skip the set and do a linked list if you can guarantee no insert repetition.

Feb 27 2021, 3:59 AM

Feb 24 2021

nani added a comment to rP23939: Switch Medium and Normal map sizes..

Maybe change default map size for random maps to "Medium" then?

Feb 24 2021, 4:34 PM

Feb 11 2021

nani added a comment to D3532: Pick map in map browser upon hitting return..

Ah, I'm not sure how to fix that, @nani.

Feb 11 2021, 12:37 AM

Feb 7 2021

nani accepted D3536: Debounce gamesetup network messages to avoid lag..
Feb 7 2021, 4:50 PM
nani added a comment to D3536: Debounce gamesetup network messages to avoid lag..

Works perfectly now.

Feb 7 2021, 4:50 PM
nani added a comment to D3536: Debounce gamesetup network messages to avoid lag..

Tested it, now there is no lag for clients. You can lower it to 200 ms so is more responsive (the lag starts when you send more than 30 network messages per second so 200 ms >> 1000/30 ~ 33.33 )
The host still lags as it updates his GUI state on its own so it should have another debounce for it somewhere (idk where but the profiler (F11) says updateGameAttributes is the culprit) for that and a debounce of 50 ms should be fine.

Feb 7 2021, 1:57 PM
nani added a comment to D3532: Pick map in map browser upon hitting return..

Should'va ve said sooner :(

Feb 7 2021, 12:22 PM
nani added a comment to D3532: Pick map in map browser upon hitting return..

I don't know if I clarified enough in the forum post but what I meant was having the confirmation hotkey trigger when you press enter while searching with the text input functionality and if you press Enter then it selects the first result for you. The idea was that when you are doing a search by name you don't need to leave/move your hands from the keyboard and directly hit enter without the need of mouse interaction and then leave the case for mouse interaction for when doing a more visual search exploration.

Feb 7 2021, 12:19 PM
nani requested review of D3533: Less interline space for lobby and gamesetup chats.
Feb 7 2021, 10:42 AM

Jun 11 2020

nani added a comment to D2805: [WIP] Reinventing React - Reducing the barrier between JS GUI and C++ GUI objects.
  • nani said there was something about possible Z issues

@nani < input?

Jun 11 2020, 10:14 PM

Jun 7 2020

nani awarded D2710: DiskPlacer: avoid scanning whole map a The World Burns token.
Jun 7 2020, 6:14 AM

Jun 3 2020

nani added inline comments to D2770: Clean up the data structures in RangeManager.
Jun 3 2020, 10:41 PM

May 25 2020

nani added a comment to D2667: Upgrade engine to handle up to 30 players.

Fair enough. Then I pass the commander to you if you want to try the grid approax.

May 25 2020, 8:19 AM
nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
May 25 2020, 4:32 AM
nani added a comment to D2667: Upgrade engine to handle up to 30 players.

std::array bool doesn't do bitpacking and its size must be known at compile time ( < c++2020 ), std::vector allocates in the heap. current diff has many more chances than a more extensive refactor + upgrade to have less bugs. Is practically guaranteed same performance as previous because it doesn't leave to chance the optimisations that the compiler should make (is basically a wrapper of some operations and data encapsulation). So I don't see an extra benefit changing current diff to using grid.

May 25 2020, 4:28 AM

May 17 2020

nani edited reviewers for D2667: Upgrade engine to handle up to 30 players, added: wraitii; removed: Restricted Owners Package.

Adding wraitii as it might interest him.

May 17 2020, 11:42 PM

Apr 14 2020

nani added a comment to D2684: [gameplay] make rams less effective vs organic units.

Not sure of this change. All the other siege units also do a similar damage to regular units and the cost is the same and this change would mean a great disadvantage compared to other siege units. Also in this case the bonus should also apply against other siege units.

Apr 14 2020, 11:14 PM

Apr 8 2020

nani added inline comments to D2663: Colorize username in chat when is ready and leaves match.
Apr 8 2020, 8:35 PM
nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
Apr 8 2020, 7:18 PM
nani updated the diff for D2667: Upgrade engine to handle up to 30 players.

Inline fixes.

Apr 8 2020, 7:14 PM

Apr 5 2020

nani added inline comments to D2069: Syntax cleanup of GUI-interface and test..
Apr 5 2020, 7:17 PM

Apr 1 2020

nani updated the diff for D2667: Upgrade engine to handle up to 30 players.

Added more parameters to the class template to specify how many bits per "value" we use.
Fixed all the code style issues and inline comments Stan and Angen mentioned.

Apr 1 2020, 9:11 PM

Mar 29 2020

nani updated the diff for D2667: Upgrade engine to handle up to 30 players.
Mar 29 2020, 5:16 PM
nani added a comment to D2667: Upgrade engine to handle up to 30 players.
In D2667#112888, @Angen wrote:

Aside from this I got idea how this could be unlimited number of players.
You would need to store inside PlayerData an array of m_data and work with the value based on index you get.
Lets say you store m_data as u32. So if you get index 0 - 16 you get first item from array. If you get index from 17 - 32 you get second item from array and subtract 16 from index.

Mar 29 2020, 5:04 PM
nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
Mar 29 2020, 5:00 PM

Mar 28 2020

nani updated the diff for D2667: Upgrade engine to handle up to 30 players.
Mar 28 2020, 7:03 PM
nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
Mar 28 2020, 7:00 PM

Mar 27 2020

nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
Mar 27 2020, 8:03 PM
nani awarded D2667: Upgrade engine to handle up to 30 players a The World Burns token.
Mar 27 2020, 7:57 PM
nani added inline comments to D2667: Upgrade engine to handle up to 30 players.
Mar 27 2020, 7:53 PM
nani updated the diff for D2667: Upgrade engine to handle up to 30 players.
Mar 27 2020, 7:52 PM

Mar 26 2020

nani updated the diff for D2667: Upgrade engine to handle up to 30 players.

Removed const and gave the child template parameter a different name so it wont complain in gcc.

Mar 26 2020, 8:34 PM

Mar 25 2020

Freagarach awarded D2667: Upgrade engine to handle up to 30 players a Yellow Medal token.
Mar 25 2020, 7:15 AM
nani updated the summary of D2667: Upgrade engine to handle up to 30 players.
Mar 25 2020, 12:47 AM
nani created D2667: Upgrade engine to handle up to 30 players.
Mar 25 2020, 12:44 AM

Feb 28 2020

nani added a comment to rP23484: Adds anti-aliasing option with FXAA algorithm..

Hmm, is it intended, that FXAA is not used on maps with PostEffect to default even when I enabled FXAA in the options?
I guess we will definitely get some reports because of that after the release...

Feb 28 2020, 6:36 PM

Feb 26 2020

nani added inline comments to D2642: Contrast-Adaptiv-Sharpening pass.
Feb 26 2020, 4:45 PM

Feb 21 2020

nani awarded D2642: Contrast-Adaptiv-Sharpening pass a Like token.
Feb 21 2020, 12:02 AM

Feb 16 2020

Angen awarded D2638: Optimize the event broadcast a Yellow Medal token.
Feb 16 2020, 1:26 PM
nani created D2638: Optimize the event broadcast.
Feb 16 2020, 1:25 PM

Feb 12 2020

nani updated the summary of D1703: Map browser for gamesetup.
Feb 12 2020, 10:58 PM