Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Profiling chunks in latest 1.12.2 compat version #89

Closed
Electrofried opened this issue Nov 26, 2017 · 10 comments
Closed

Profiling chunks in latest 1.12.2 compat version #89

Electrofried opened this issue Nov 26, 2017 · 10 comments

Comments

@Electrofried
Copy link

Electrofried commented Nov 26, 2017

I am trying to use tick profiler on 1.12.2, it is working fine, but I can't seem to get it to profile just one chunk anymore, although I am sure this worked with past versions. Checked all the options that /profile lists and none of them seem to list the ability to restrict profiling to one chunk.
The wiki lists /profile c (and this is what I remember using) however that gives entity counts now and can also only be restricted to dimensions.
It was super useful to profile a dimension to find the worst chunks then restrict it to that chunk and just profile those in order to track down issues. Is this feature removed/coming back or am I just being special?

@xorinzor
Copy link

xorinzor commented Jan 8, 2018

Can confirm, /profile c is entity count, not Chunk information.

Additionally, there is no way using /profile e to tell what chunk contains the most items, or where items are generally coming from.

@ArcanoxDragon
Copy link

Can confirm this as well, plus /profile e does not accept the time= parameter like it should (all values still result in 30 seconds of profiling) and it does not measure TileEntities like the source code implies it should.

@Fatmice
Copy link

Fatmice commented Jan 29, 2018

Can confirm this as well, plus /profile e does not accept the time= parameter like it should (all values still result in 30 seconds of profiling) and it does not measure TileEntities like the source code implies it should.

How did you issue it?
/profile e 60 will profile for 60s
/profile e 60 0 will profile overworld for 60s
/profile e 60 0 10 will profile overworld for 60s and list 10 things

@LunNova
Copy link
Member

LunNova commented Feb 9, 2018

profile c is now for entity counting. This is intentional and was changed as part of 95a252e which standardised profile command handling.

I was planning to add single chunk profiling as an option for profile e, it seemed more logical than a separate command. Unfortunately I haven't had much time to work on TickProfiler recently and have not yet added this.

@tolly765
Copy link

We can wait, take your time nallar 😄

@LunNova LunNova closed this as completed in 17952a5 Mar 1, 2018
@LunNova
Copy link
Member

LunNova commented Mar 1, 2018

Use /profile e chunk=current in the latest version

@Dimentive
Copy link

@nallar That command doesn’t work. Has this been changed?

@Electrofried
Copy link
Author

/profile e chunk=current

@LunNova
Copy link
Member

LunNova commented Mar 17, 2019

Has this been changed?

always been wrong, i messed up in that comment. 😢

@Dimentive
Copy link

No worries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants