RPG Minimap v1.4

[kokon]

Ehm, if map is generated via code, will it be rendered somehow?
I think this is not possible

 

[Antares]

Ehm, if map is generated via code, will it be rendered somehow?
I think this is not possible

It will be with a few changes.

 

[kokon]

But how do you rendern terrain to minimap if there was no terrain?

 

[Antares]

You still have normal terrain after you generate it? So, just initialize the minimap after the terrain has been generated. Or are you doing some crazy shit where you're not using normal terrain at all?

 

[kokon]

I thought you can not do it after game started

 

[Antares]

Updated to Version 1.1

• Added the option to initialize RPG Minimap manually.
• Added the RedrawRect function, which updates the minimap textures in the specified area.

 

[kokon]

Hmmm, i'm using jass...
As far as i understand this library is the only way to redraw a minimap in wc3 if I textured it dynamically during first 0.5 seconds of the game

 

[Antares]

Sorry, the system is Lua-only (with a Lua-GUI port coming eventually).

 

[kokon]

Lua-GUI plz? )

 

[Antares]

Lua-GUI plz? )

Didn't you just say you were using JASS?

 

[kokon]

I think ability to redraw map is supervaluable. If it was available in jass... see...

 

[Xthreo]

Absolutely Amazing, we needed this!

 

[Antares]

Absolutely Amazing, we needed this!

Thank you! Let's make some ambitious RPG maps!

I think ability to redraw map is supervaluable. If it was available in jass... see...

Alright, I'll work on it after my next update/when @Wrda approves it. Don't have to do the changes twice then.

 

[Wrda]

Can't wait for the next 10k lines system after this one

 

[Wrda]

--[[
    
Set the values for size, texture, visibilityRadius and level for the minimap icons of different widget types (fourCC codes).


    Size                        The size of the minimap icon in screen coordinates.
    Texture                     The path to the texture for the minimap icon.
    VisibilityRadius            If set, the minimap icon will only be visible up to the specified distance from the camera. Set to -1 for no visibility radius.
    Level                       The level of the created frame. Frames with a higher level occlude frames with a lower level. Setting the level to a negative number
                                or to a value too high will crash the game.
    ]]


    MINIMAP_WIDGET_TYPE_TEXTURE                 = {
        Etyr = "TyrandeMinimap.blp",
        phea = "HealingPotionMinimap.blp"
    }
    MINIMAP_WIDGET_TYPE_SIZE                    = {
    }
    MINIMAP_WIDGET_TYPE_VISIBILITY_RADIUS       = {
    }
    MINIMAP_WIDGET_TYPE_LEVEL                   = {
    }
    MINIMAP_WIDGET_TYPE_COLOR                   = {
    }

I believe this is rather awkward to expect the user to add widget types this way, even if they can be declared anywhere, when the list comes big it's just a mess to work with. Why not have some sort of a function such as
RPGMinimap.RegisterWidgetType(fourCCString, texture, size, visibility, level, colour)? Seems faster, clear and simpler. Obviously the tables can the remain as they are.

local char = string.char

Unused at the moment.

local function DrawMinimapIconOnReset and InitDrawWaypointMarker have unused parameters, but this definetely must be because of the pairs done in ALICE, I presume.
Unused minimapArtist parameter.

local index = (math.floor((bj_RADTODEG*angle) / WAYPOINT_MARKER_ANGLE_SPACING + 0.5))*WAYPOINT_MARKER_ANGLE_SPACING

Lua ad: Hey friend! Are you tired of using BJs and JASS pain and nonsense? Try math.deg(radians) and it will make your day bright! It's on a 95% sale, don't miss your shot! Best regards from Lua Cult!

---@param translocator Translocator
    ---@param drawable any
    ---@data table
    local function TranslocateMinimapIconOnReset(translocator, drawable, data)
        if translocationZoneOf[drawable] == translocator then
            translocationZoneOf[drawable] = nil
            translocatedCloneOf[drawable] = nil
        end
        ALICE_Kill(data.clone)
        data.clone = nil
    end

Missed "param" on annotation

Another powerful resource that expands the horizons and let people dream. Just make that a function to register the widgets easier and then it's instant upgrade!

Approved

 

[Antares]

I believe this is rather awkward to expect the user to add widget types this way, even if they can be declared anywhere, when the list comes big it's just a mess to work with. Why not have some sort of a function such as
RPGMinimap.RegisterWidgetType(fourCCString, texture, size, visibility, level, colour)? Seems faster, clear and simpler. Obviously the tables can the remain as they are.

I think you're misunderstanding the purpose of these tables. You don't have to set the values for every widget type. They are just the types for which the configurations you set further up shouldn't apply. They are also only one of the many options you have to control the properties, as you can write anything into the functions further down. Nonetheless, a function to modify widget types is a good idea and I wrote it such that it can be called even after icons are already being drawn.

        ---Assigns to all widgets with the specified code the properties in the provided table. The possible keys of the property table are texture, size, level, visibiltiyRadius, color, and exception (which can be true or false).
        ---@param fourCC string | integer
        ---@param propertyTable table
        ModifyWidgetType = function(fourCC, propertyTable)
            fourCC = type(fourCC) == "string" and fourCC or string.pack(">I4", fourCC)
            local key = minimapPreInit and FourCC(fourCC) or fourCC

            if propertyTable.texture then
                MINIMAP_WIDGET_TYPE_TEXTURE[key] = propertyTable.texture
            end
            if propertyTable.size then
                MINIMAP_WIDGET_TYPE_SIZE[key] = propertyTable.size
            end
            if propertyTable.level then
                MINIMAP_WIDGET_TYPE_SIZE[key] = propertyTable.level
            end
            if propertyTable.visibilityRadius then
                MINIMAP_WIDGET_TYPE_SIZE[key] = propertyTable.visibilityRadius
            end
            if propertyTable.color then
                MINIMAP_WIDGET_TYPE_SIZE[key] = propertyTable.color
            end
            if propertyTable.exception then
                ---@diagnostic disable-next-line: assign-type-mismatch
                MINIMAP_WIDGET_TYPE_EXCEPTIONS[key] = propertyTable.exception
            end

            if minimapInitialized then
                ALICE_ForAllObjectsDo(ClearAllCaches, fourCC)
                if propertyTable.exception == true then
                    ALICE_ForAllObjectsDo(AddDrawableIdentifier, fourCC)
                elseif propertyTable.exception == false then
                    ALICE_ForAllObjectsDo(RemoveDrawableIdentifier, fourCC)
                end
            end
        end,

local function DrawMinimapIconOnReset and InitDrawWaypointMarker have unused parameters, but this definetely must be because of the pairs done in ALICE, I presume.
Unused minimapArtist parameter.

Yes, the argument order in ALICE interaction function calls is (male, female) and (male, female, dataTable) in all callback functions.

local index = (math.floor((bj_RADTODEG*angle) / WAYPOINT_MARKER_ANGLE_SPACING + 0.5))*WAYPOINT_MARKER_ANGLE_SPACING

Lua ad: Hey friend! Are you tired of using BJs and JASS pain and nonsense? Try math.deg(radians) and it will make your day bright! It's on a 95% sale, don't miss your shot! Best regards from Lua Cult!

Why do you want me to make my code slower?

---@param translocator Translocator
    ---@param drawable any
    ---@data table
    local function TranslocateMinimapIconOnReset(translocator, drawable, data)
        if translocationZoneOf[drawable] == translocator then
            translocationZoneOf[drawable] = nil
            translocatedCloneOf[drawable] = nil
        end
        ALICE_Kill(data.clone)
        data.clone = nil
    end

Missed "param" on annotation

Good catch.

Another powerful resource that expands the horizons and let people dream. Just make that a function to register the widgets easier and then it's instant upgrade!

 

[Wrda]

I think you're misunderstanding the purpose of these tables. You don't have to set the values for every widget type. They are just the types for which the configurations you set further up shouldn't apply. They are also only one of the many options you have to control the properties, as you can write anything into the functions further down.

For some reason I thought MINIMAP_WIDGET_TYPE_EXCEPTIONS was seperate from the other tables...brainfart.
That's fine then.

Why do you want me to make my code slower?

It's like learning a musical instrument, if you can do it fast, you can do it slow Lua you failed me this time

This deserves a place in the sky.

High Quality

 

[Antares]

Updated to v1.3:

• Added the CAMERA_IS_EXPLORER config option. The minimap area around the camera will only be revealed automatically if this option is set.
• Added the RPGMinimap.SetExplorer function, which causes an object to reveal the minimap around itself. This can be used to share exploration between players and makes RPG Minimap fully multiplayer viable!
• Changed the RPGMinimap.ModifyWidgetType function documentation, clarifying that it only modifies the config tables and won't have an effect unless their values are used.
• Updated ALICE to 2.4.2.

For some reason I thought MINIMAP_WIDGET_TYPE_EXCEPTIONS was seperate from the other tables...brainfart.
That's fine then.

It is different because it is used in the internals, unlike the other tables.

This deserves a place in the sky.

High Quality

Thank you! (147, 101, 184), good color!

 

[The Nightmare Book]

Cool, cool stuff.

 

[Antares]

Version 1.4

• Added an option to use a static minimap that renders the entire area.