the-international-screeps-bot / the-international-open-source Goto Github PK

If a scout finds a room it wants to sign but a reserver is there that "unsigns" it immedietly the scout will get caught in a loop of signing every 2 ticks.

https://github.com/CarsonBurke/The-International-Screeps-Bot/blob/Typescript-(Main)/src/room/creeps/roleManagers/international/scoutManager.ts#L40

MFR (madeForRoads) memory property applied to creeps that should weight roads in costMatrix pathing

Builders should be spawned with more work and carry and less move. They should wait for haulers to come to them and not move.

At low RCL when there is no sourceContainer, sourceHarvesters should have their carry capacity used for reservations by haulers to avoid energy decay from it being dropped on the ground

Things to consider when improving code:

• Should probably only safemode when a spawn is at risk.
• Spawns are probably not at risk if there is a sufficiently HP rampart on top of them
• Spawns can be killed in one hit by sufficiently fitted dismantle creeps, so checking if spawn hits are low isn't a good idea
• Perhaps check enemy range to spawn and spawn rampart existence for each spawn each tick

Have a method in the InternationalManager that identifies the best new claimRequest for all communes and submits it

sourceHarvestPositions should be stored in memory in the following structure

sourceHarvestPositions = [
packedSourcePositions // one packed array of positions (sorted by range to source path)
]

There should be a room getter to enable easy access to the harvest positions. The result should be:

sourceHarvestPositions = [
[RoomPosition] // one array of unpacked room positions in previous sort order per source
]

An error occured while upgrading to latest version

Error:
TypeError: Cannot read property 'source1RemoteHarvester' of undefined at RemoteHarvester.preTickManager (main:1269:61) at InternationalManager.creepOrganizer (main:1342:15) at InternationalManager.run (main:962:10) at Object.loop (main:9166:26) at __mainLoop:1:52 at __mainLoop:2:3 at Object.exports.evalCode (<runtime>:15845:76) at Object.exports.run (<runtime>:46468:24)

Harvester looks in commune.creepsFromRoomWithRemote for its remote name but it could not be found because that room isn't being remote mined currently

Code:
`RemoteHarvester.prototype.preTickManager = function () {
if (!this.memory.remoteName) return

 const role = this.memory.role as 'source1RemoteHarvester' | 'source2RemoteHarvester'

 // Reduce remote need
 customLog('REMOTE HARVESTER CHECK', JSON.stringify(this.parts))
 Memory.rooms[this.memory.remoteName].needs[remoteNeedsIndex[role]] -= this.parts.work

 const commune = Game.rooms[this.memory.communeName]
 if (!commune) return

 // Add the creep to creepsFromRoomWithRemote relative to its remote

 console.log(commune.name, commune.creepsFromRoomWithRemote[this.memory.remoteName], this.memory.remoteName)
 commune.creepsFromRoomWithRemote[this.memory.remoteName][role].push(this.name)

}`

Logs: E72N6 undefined E72N5

*This was fixed by resetting memory using CleanMemory

Remote isnt being defended, keeps sending normal creeps instead of defenders

Please check conditions and make the spawn prio lower.

For example, at low RCL and closed off sources, haulers will be over-spawned relative to the actual income from the source

Additionally, have upgraders use the controllerLink even when there is no hubLink.

This improvement requires the pre-requisite of hauler reservation instead of tasks.

Likely the result of haulers trying to path to a Coord that they think is in their room, but is actually in the commune room they are bouncing into. Unkown specific fix

TypeError: Cannot read properties of undefined (reading 'x') at basePlanner:1847:41 at manageBasePlanning:1968:48 at constructionManager:1964:5 at Object.communeManager [as commune]:2130:5 at roomManager:2994:51 at Object.loop:3040:5 at __mainLoop:1:22694 at eval:2:4
Constantly getting this error on spawn in training. Unsure about in a actual server if error persists. This error starts instantly after start. It managed to place one construction site down before.

Node v12.22.10 used to compile, not sure if that effects anything.

Edit: nvm im stupid, ignore me please uwu
Edit2: Your scouts are still dumb tho

Clearly, this builder with 100 store capacity doesn't need all these haulers trying to transfer... NEGATIVE ENERGY to it.

As you can see, the tickConfig and creepOrganizer take a lot of CPU while performing (I think) no intents. Therefore, there are lots of CPU optimizations we can make to these functions!

/src/room/linkFunctions.ts

Either comment or maths is wrong.
Comment suggests half but maths of *0.25 is quarter.

lines 25 || 27

`
// If the link is more than half full, iterate

        if (receiverLink.store.energy > receiverLink.store.getCapacity(RESOURCE_ENERGY) * 0.25) continue

`