Factorio Forums

Adil wrote:Good job.
I wanted to do that too.
Can we have a discussion?

Adil wrote: I didn't know create_entity can spawn colliding entities together, probably that will get fixed someday.

Adil wrote: You can group events with same handlers together like this:

Code: Select all

game.on_event({defines.events.on_built_entity,defines.events.on_robot_built_entity},function(event) ... end)

Adil wrote:Speaking of which on_robot_built_entity handler is bugged as it lacks the line 7 from on_built_entity.

Adil wrote:Both tooltips can be made accessible by changing selection boxes of entities. Say logistic container might have selection_box = {{-1, -1 }, {0, 1}}, and the gun - {{0,1 }, {1, 1}}.

Adil wrote:Same goes to collision boxes. In a fact I'd make the gun non-colliding at all, by setting collision_mask={}.
And probably make chest invincible while gun lives instead of constantly updating its health.

Adil wrote:As a continuation of previous, I'd say that this tandem creation should be initiated by placing the logistic container rather than by placing gun turret. That way blueprints would preserve requester conditions (but all item tooltips would not provide gun-power info).
Alternatively, requester might be made into standalone module that can be placed over turret (like my unloader mods).

Adil wrote:Also, is it really necessary to sync every turret-chest pair sixty times a second?

Afforess wrote: Absolutely! No reason you can't contribute to the project on github, btw.

Afforess wrote: hide the gun while showing it's tooltip, then there will be no reason to have the gun turret placed first anymore.

Afforess wrote: If I made it update only update every 5 or 10 ticks, then I would run into the possibility that health or ammo would become out of sync enough that the turret could die,

Afforess wrote: or ammo run out.

require 'util'
for _,tech in pairs(data.raw.technology) do
    if tech.effects then
        for _,effect in pairs(effects) do
            if effect.turret_id and effect.turret_id=='gun-turret' then
                local neweff=table.deepcopy(effect)--defined in util
                neweff.turret_id='gun-turret-2'
                table.insert(effects,neweff)
            end
        end
    end
end

Xterminator wrote:Really cool idea! Having a turret with a built in Logistic request slot is brilliant, and surprised it hasn't been done yet. The ammo boxes are pretty awesome too, especially for turrets.

Mind if I do a mod spotlight on this in the near future?

Adil wrote:

Afforess wrote: Absolutely! No reason you can't contribute to the project on github, btw.

Yeah, one day I will get around registering there.

Afforess wrote: hide the gun while showing it's tooltip, then there will be no reason to have the gun turret placed first anymore.

Tooltip is shown when you select entity. I doubt you can get it from completely hidden object. Suggested selection boxes would just allow you to select chest when hovering cursor over the left side of entity, and gun when over right.

Adil wrote: Not that I'd see that as a problem (provided it is not happenning each time turret shoots).
Turret shoots 10 bullets/second, that's 1 clip/second, updates hardly need to happen more often than that, though then it'll need to load more than a single clip per check. With bullet speed upgrades it wastes 2 clips/sec but it's not far-fetched assumption that by the time ammoboxes would be used. (Though I myself am usually against such items.)

Adil wrote: By the way, your turrets are not affected by the gun-turret upgrades and thus vanilla overpower them after all upgrades.
The following would put them in upgrades.

data-upgrades.lua

Code: Select all

require 'util'
for _,tech in pairs(data.raw.technology) do
    if tech.effects then
        for _,effect in pairs(effects) do
            if effect.turret_id and effect.turret_id=='gun-turret' then
                local neweff=table.deepcopy(effect)--defined in util
                neweff.turret_id='gun-turret-2'
                table.insert(effects,neweff)
            end
        end
    end
end

Adil wrote: but it's not far-fetched assumption that by the time ammoboxes would be used. (Though I myself am usually against such items.)

Xterminator wrote:Really cool idea! Having a turret with a built in Logistic request slot is brilliant, and surprised it hasn't been done yet. The ammo boxes are pretty awesome too, especially for turrets.

Mind if I do a mod spotlight on this in the near future?

capthavic wrote:Update today broke the mod. Any chance of getting an patch for it?

Darkestnoir wrote:

Code: Select all

Error while running the event handler: __AmmoBox__/control.lua:103: attempt to call field 'get_inventory' (a nil value)

Getting this when i load my 12.13 save...

Darkestnoir wrote:

Code: Select all

Error while running the event handler: __AmmoBox__/control.lua:103: attempt to call field 'get_inventory' (a nil value)

Getting this when i load my 12.13 save...

Darkestnoir wrote:

Code: Select all

Error while running the event handler: __AmmoBox__/control.lua:133: petroleum-generator doesn't have an inventory with index(1)

A new "Game Breaking Bug", with 12.14

Darkestnoir wrote:

Code: Select all

Error while running the event handler: __AmmoBox__/control.lua:133: petroleum-generator doesn't have an inventory with index(1)

A new "Game Breaking Bug", with 12.14

• Add compressed flamethrower ammo
  • Holds 100x magazine size of standard flame thrower ammo
  • Has 2x attack distance of regular flamethrower ammo
  • Requires Flamethrowers 2 tech to unlock
• Added flamethrower 2 tech to unlock compressed flamethrower ammo