Duplication of villager
attilavv opened this issue ยท 6 comments
Minecolonies version
Forge 2604 ( in modpack )
version ; minecolonies-universal-1.12.2-0.8.7273
- Barbarian is disabled in config file
Expected behavior
- No duplication
Actual behaviour
- Trace of duplication.
1 - In fml-junk-earlystartup.log ( extract )
fml-junk-earlystartup.log
3 - Old picture in old post.
Steps to reproduce the problem
Not found a process for reproduct at this time.
Hi,
Server upgraded : minecolonies-universal-1.12.2-0.8.7312
Now : I have 2 deliveryman from same house. And one group of 4 of soldiers have duplication for 3 soldiers.
For me it is "always" (99%) on upgrade and "some time" in "start up" (1%).
YES i make many and many upgrade for found some bug. Like a big memory leak ( 2GB to 6GB in 2/3 hours ). Removing some mods again and again.
And YES for me you have resolved this big memory leak.
But at this time i update only Minecolonies mod. Same time other for minor change like some recipes.
We really need to work on entity deduplication. I will take a look today to start using WeakReferences and Optionals.
@Raycoms Ping me in Slack.
Not sure for me ...
Before the big washing of debug i can say memory used = disk space.
Are you open a windows console in start up ?
A console Java not in the cycle of Forge. Only a little debug console and when you add a message it is output on Forge fml-junk-earlystartup.log. A console in a separated tread ? But i have a high CPU usage. And CPU usage change when i change version of Minecolonies. A console that wait the Forge tread and take CPU ressource for nothink. A console is same problem when you run a JAR, you can have all history and it is bad.
In fact all CPU ressource is "for me" doubled.
With this version ( 7312 ) i use 60% CPU ( indice is Verygame CPU usage )
with 7273 ... 90% ( the beging of remove debugging message )
with 7200 ... 120% or more. Verygames stop server after 3 reboots in 2 hours for High CPU usage.
Before ... it is better ... 60% ...
Read my other issue for chunks loading.
For me, all is linked. Memory leak or usage, High CPU usage, Chunk loading, some fonction out the tread of Forge and duplication but it is a sign that some register, byte or other is not full controled.
If i have time, tomorrow i open a issue for proposion. For exemple, stop add news villager but make a price for recrute on town hall. That stop all kill by "lava bucket" for bad stat and possible to clean all villager for debugging. Add redstone control for "place in pause" a building if bug, for testing or debugging and for game play too ... stop farmer, stop wood ...
xD
Server is .. OK ( 2x 60% ) ..
Client is very out.1 FPM = Frame Per Minute !
I don't know how many villager is present but it is many and many.
Server is "open" from 8 hours without reboot ( 40/60% it is acceptable for Verygames )
Memory 3250MB to 3850MB ( for 8 Hours without connection, no player have make connection )
E = 259 entity in render for a total of 5765 ! ! !
Add a config entry for reset all villager.
ResetAtNextReset = True / False.
After reset is set to True place it to False.
Add a command for "disable", or fire like a job for village all or some villager.
At this time it is a Alpha / Beta this command is needed.
After for a Release version remove it if you want.
On Wiki add a description for Config and all command and how it work.
But for a problem like this the best way is the Wiki and found a command or a soluce.