System PUI - Perfect Unit Indexing

Jesus4Lyf

Good Idea™
Reaction score
397
Jesus4Lyfe, what in the world could you want from unit indexing that neither PUI nor AutoIndex does?
AIDS structs, of course. :)
It is imperative that indexes always be as low as possible because of the way jass arrays allocate memory.
>I believe in "don't fix if it is not broken" phylosophy.
>PUI works as it should and it works well, I have no intention of wasting my time on maybe-improvements.

What are you talking about? PUI is broken as hell by your logic. And it DOESN'T free indexes after 5 seconds. See demo map. Your algorithm vs mine.

By the way, that's only the average index. Would you like to see the max?
Edit: If so...
JASS:
library TestPUI initializer test uses PUI, Mod
    private function periodic takes nothing returns nothing
        globals
            private integer PUI_Max=0
            private integer Mod_Max=0
            private integer Index
            
            private unit u
        endglobals
        
        set u=CreateUnit(Player(0),'hfoo',0,0,0)
        set Index=GetUnitIndex(u)
        if Index>PUI_Max then
            set PUI_Max=Index
        endif
        call RemoveUnit(u)
        
        set u=CreateUnit(Player(0),'hfoo',0,0,0)
        set Index=Mod_GetUnitIndex(u)
        if Index>Mod_Max then
            set Mod_Max=Index
        endif
        call RemoveUnit(u)
        
        call BJDebugMsg("Max Indexes: PUI: "+I2S(PUI_Max)+" | Modded: "+I2S(Mod_Max)+".")
    endfunction
    private function test takes nothing returns nothing
        call TimerStart(CreateTimer(),0.02,true,function periodic)
    endfunction
endlibrary

Shows that PUI with your algorithm breaks when my algorithm is only on about index 3300. (Yes, 8100 to 3300, roughly.)
 

Attachments

  • test3.w3x
    24.3 KB · Views: 374

Cohadar

master of fugue
Reaction score
209
What are you talking about? PUI is broken as hell by your logic. And it DOESN'T free indexes after 5 seconds. See demo map. Your algorithm vs mine.
It frees indexes after 5-10 seconds, big deal.
PUI was the first indexing system out there, and by my logic it was and still is the best indexing system ever made.
A lot of really good map makers seems to agree with me on that one too.

I really don't care about your test maps because I have previous experience with you tiliting the results in your favour (TT vs KT2 thing)
so I am not going to waste any more of my time with this.

Please go develop your system in your own thread.
 

Jesus4Lyf

Good Idea™
Reaction score
397
I really don't care about your test maps because I have previous experience with you tiliting the results in your favour (TT vs KT2 thing)
Don't even try that. We ran your tests and had the same results as mine.

If I say PUI is broken, you know what I'm talking about. Your index recycling is wrong, as shown in the above test maps.

All the same, thanks for your time. I do value it more than a couple of nanoseconds from my computer. :)
 

kingkingyyk3

Visitor (Welcome to the Jungle, Baby!)
Reaction score
216
Huh? I downloaded the map and put my unit indexing system as competitor. At early stage, moded PUI had lowest value.. But after 1 minute, I found my one has lowest value! lol. I had same decay time, recycler period....
 

Romek

Super Moderator
Reaction score
964
@kingkingyyk3:
Chances are, your indexing system is crap.
 

Cohadar

master of fugue
Reaction score
209
If I say PUI is broken, you know what I'm talking about. Your index recycling is wrong, as shown in the above test maps.
Ye you are talking about imaginary world in your head.

Because what you say must be true and the fact that me and Builder Bob have tested PUI extensively and that PUI had been used in all kinds of maps for almost 4 years without any errors/performance issues does not really matter.

It also does not matter that I explained to you (for the third time now) that optimizing property A of the system does not necessarily make the system better because it can fuck up properties B and C and D...

It also does not matter that you make your test maps in such a way as to compare one property of the system while disregarding others (intentionally) and thus giving a false impression of the entire sys.

Really I don't get why people who make modified/alternate versions of my systems always feel the need to prove to me that they are better somehow.
Prove it to yourself and prove it to the people who might want to use it and leave me alone.

If I see anyone else talking about their indexing system in my thread I am reporting him for trolling.
 

kingkingyyk3

Visitor (Welcome to the Jungle, Baby!)
Reaction score
216
kingkingyyk3 owns PUI?

No, that test map was a demonstration of recycling algorithms, not unit indexing systems.

Actually, if you recycled faster than Modded PUI, on the same period, then your recycler is probably doing too much work. :p

My system only went up to 255 when PUI was full.

My recycle here. (similar with TT callback) :p
JASS:
        if GetUnitUserData(INDEXED_UNITS[Index]) == 0 then
            if DECAY_TICK[Index] > 0 then
                set DECAY_TICK[Index] = DECAY_TICK[Index] - 1
            else
                set INDEXED_UNITS[Index] = INDEXED_UNITS[COUNTER]
                set DECAY_TICK[Index] = DECAY_TICK[COUNTER]
                set INDEXED_UNITS[COUNTER] = null
                set DECAY_TICK[COUNTER] = 0
                set COUNTER = COUNTER - 1
                set LAST_FREE_INDEX = COUNTER
            endif
        endif
//Crap recycling <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite8" alt=":D" title="Big Grin    :D" loading="lazy" data-shortname=":D" />
 

Jesus4Lyf

Good Idea™
Reaction score
397
@kingkingyyk3
Your system is broken, you recycle more than 32 indexes per second.
Your code is horrible. :( You should switch to a real system, it will do significantly less processing in the background, and you don't have to write it yourself.

freeride.jpg


Also, your code has nothing to do with TT.
 

Joker(Div)

Always Here..
Reaction score
86
There must be a bug of some sort b/c I am getting units with the same index and the CHECK value also becomes bugged with it.
JASS:
scope Checkpoint initializer Init

    //! runtextmacro PUI_PROPERTY( &quot;&quot;, &quot;integer&quot;, &quot;CHECK&quot;, &quot;-1&quot; )

    globals
        private constant integer MAX_CHECKPOINTS = 8
    endglobals
    //======================================================
    
    globals
        private integer array CHECKPOINTS[MAX_PLAYERS] 
        
        public rect array AREA[MAX_CHECKPOINTS]
    endglobals
    //======================================================
    
    private function Conditions takes nothing returns boolean
        return CHECK[GetTriggerUnit()] &gt; -1
    endfunction
    
    private function Actions takes nothing returns nothing
        local unit a = GetTriggerUnit()
        local player p = GetOwningPlayer( a )
        local integer c = CHECK[a] + 1
        local real x = GetRectCenterX( AREA[c] )
        local real y = GetRectCenterY( AREA[c] )
        
        if IsPointInRegion( GetTriggeringRegion(), x, y ) then
        
            if (c + 1) &gt;= MAX_CHECKPOINTS then
                set c = GetPlayerId( p )
                set CHECKPOINTS[c] = CHECKPOINTS[c] + 1
                call BoardUpdateText( 2, c + 1, &quot;  &quot; + I2S( CHECKPOINTS[c] ) )
                
                set c = 0
                set CHECK[a] = c
                set x = GetRectCenterX( AREA[c] )
                set y = GetRectCenterY( AREA[c] )
            endif
            
            set CHECK[a] = -1
            call ShowUnit( a, false )
            call SetUnitExploded( a, true )
            call UnitApplyTimedLife( a, &#039;BTLF&#039;, 1. )
            //Pretty sure its PUI b/c it does not bug if I do not kill the entering unit.
            
            set a = CreateUnitEx( p, x, y )
            set CHECK[a] = c
            
            if p == GetLocalPlayer() then
                call ClearSelection()
                call SelectUnit( a, true )
                call PanCameraToTimed( x, y, 0. )
            endif
        endif
        
        set a = null
    endfunction
    //======================================================
    
    private function Init takes nothing returns nothing
        local integer i = 0
        
        set AREA[0] = Rect( -2624., -1888., -704.,   32. )
        set AREA[1] = Rect( -830.,   575.,   415.,   1760. )
        set AREA[2] = Rect( -4225.,  670.,  -3135.,  1790. )
        set AREA[3] = Rect( -4225., -3138., -3232., -2496. )
        set AREA[4] = Rect(  1092., -3171.,  1543., -2624. )
        set AREA[5] = Rect(  1230.,  2264.,  1550.,  2688. )
        set AREA[6] = Rect( -5056.,  2464., -4632.,  2688. )
        set AREA[7] = Rect( -5056., -3232., -4732., -3096. )
        
        loop
            call UnitEnterEvent( Condition( function Conditions ), function Actions, AREA<i> )
            set i = i + 1
            exitwhen i &gt;= MAX_CHECKPOINTS
        endloop
    endfunction

endscope</i>
 

Cohadar

master of fugue
Reaction score
209
Tell me how am I goint to reproduce your error from that script?
Please make me a demo map that demonstrates the bug.

Are you using UnitUserData directly somewhere?
 

Joker(Div)

Always Here..
Reaction score
86
Test map.

Maybe I'm doing something really weird with the unit replacement. Made a Demo of the bug.

EDIT: I found that the "BUGGED" msg gives out false-positives every once in a while when you don't kill the entering units.
 

Attachments

  • PUI BUG.w3x
    25.9 KB · Views: 371

Cohadar

master of fugue
Reaction score
209
Very strange thing, it really is an PUI error.
It was introduced with 5.2, version 5.1 works fine.

I have reattached 5.1 to first post, use that until I release 5.3
Thanks for finding this Joker(Div)

EDIT:
fixed, version 5.3 is out.
In case someone cares the problem was that I forgot to shift one index from free index section to used index section when indexes are recycled.
This was a problem because used and free indexes were kept in same array "to save space"..
This was very stupid of me because it complicated algorithm and thus caused me to miss this bug.
And I am the one that always tells people that they overdo with optimization...:banghead:
 

NetherHawk

New Member
Reaction score
26
hi.. pui only recycles indexes when a unit is removed right? that means that an index set for a hero will nvr become 0. Is that right?
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • The Helper The Helper:
    I am great and it is fantastic to see you my friend!
    +1
  • The Helper The Helper:
    If you are new to the site please check out the Recipe and Food Forum https://www.thehelper.net/forums/recipes-and-food.220/
  • Monovertex Monovertex:
    How come you're so into recipes lately? Never saw this much interest in this topic in the old days of TH.net
  • Monovertex Monovertex:
    Hmm, how do I change my signature?
  • tom_mai78101 tom_mai78101:
    Signatures can be edit in your account profile. As for the old stuffs, I'm thinking it's because Blizzard is now under Microsoft, and because of Microsoft Xbox going the way it is, it's dreadful.
  • The Helper The Helper:
    I am not big on the recipes I am just promoting them - I use the site as a practice place promoting stuff
    +2
  • Monovertex Monovertex:
    @tom_mai78101 I must be blind. If I go on my profile I don't see any area to edit the signature; If I go to account details (settings) I don't see any signature area either.
  • The Helper The Helper:
    You can get there if you click the bell icon (alerts) and choose preferences from the bottom, signature will be in the menu on the left there https://www.thehelper.net/account/preferences
  • The Helper The Helper:
    I think I need to split the Sci/Tech news forum into 2 one for Science and one for Tech but I am hating all the moving of posts I would have to do
  • The Helper The Helper:
    What is up Old Mountain Shadow?
  • The Helper The Helper:
    Happy Thursday!
    +1
  • Varine Varine:
    Crazy how much 3d printing has come in the last few years. Sad that it's not as easily modifiable though
  • Varine Varine:
    I bought an Ender 3 during the pandemic and tinkered with it all the time. Just bought a Sovol, not as easy. I'm trying to make it use a different nozzle because I have a fuck ton of Volcanos, and they use what is basically a modified volcano that is just a smidge longer, and almost every part on this thing needs to be redone to make it work
  • Varine Varine:
    Luckily I have a 3d printer for that, I guess. But it's ridiculous. The regular volcanos are 21mm, these Sovol versions are about 23.5mm
  • Varine Varine:
    So, 2.5mm longer. But the thing that measures the bed is about 1.5mm above the nozzle, so if I swap it with a volcano then I'm 1mm behind it. So cool, new bracket to swap that, but THEN the fan shroud to direct air at the part is ALSO going to be .5mm to low, and so I need to redo that, but by doing that it is a little bit off where it should be blowing and it's throwing it at the heating block instead of the part, and fuck man
  • Varine Varine:
    I didn't realize they designed this entire thing to NOT be modded. I would have just got a fucking Bambu if I knew that, the whole point was I could fuck with this. And no one else makes shit for Sovol so I have to go through them, and they have... interesting pricing models. So I have a new extruder altogether that I'm taking apart and going to just design a whole new one to use my nozzles. Dumb design.
  • Varine Varine:
    Can't just buy a new heatblock, you need to get a whole hotend - so block, heater cartridge, thermistor, heatbreak, and nozzle. And they put this fucking paste in there so I can't take the thermistor or cartridge out with any ease, that's 30 dollars. Or you can get the whole extrudor with the direct driver AND that heatblock for like 50, but you still can't get any of it to come apart
  • Varine Varine:
    Partsbuilt has individual parts I found but they're expensive. I think I can get bits swapped around and make this work with generic shit though
  • Ghan Ghan:
    Heard Houston got hit pretty bad by storms last night. Hope all is well with TH.
  • The Helper The Helper:
    Power back on finally - all is good here no damage
    +2
  • V-SNES V-SNES:
    Happy Friday!
    +1
  • The Helper The Helper:
    New recipe is another summer dessert Berry and Peach Cheesecake - https://www.thehelper.net/threads/recipe-berry-and-peach-cheesecake.194169/

      The Helper Discord

      Members online

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top