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

Paper warns about the usage of Timing(s) #5

Open
FireInstall opened this issue Jul 23, 2023 · 5 comments
Open

Paper warns about the usage of Timing(s) #5

FireInstall opened this issue Jul 23, 2023 · 5 comments

Comments

@FireInstall
Copy link

Used Version

v1.5-SNAPSHOT (build 297)

Environment description

paper 1.20

Log

[18:49:00 WARN]: [!] The timings profiler has been enabled but has been scheduled for removal from Paper in the future.
    We recommend installing the spark profiler as a replacement: https://spark.lucko.me/
    For more information please visit: https://github.com/PaperMC/Paper/issues/8948
[18:50:23 WARN]: [Pipes] Could not save lang.de.yml to plugins/Pipes/lang.de.yml because lang.de.yml already exists.
[18:50:23 WARN]: Plugin 'Pipes' is creating timing 'move' - this is deprecated behavior, please report it to the authors: Lord36, Phoenix616
[18:50:23 WARN]: Plugin 'Pipes' is creating timing '## filter' - this is deprecated behavior, please report it to the authors: Lord36, Phoenix616
[18:50:23 WARN]: Plugin 'Pipes' is creating timing '## transfer' - this is deprecated behavior, please report it to the authors: Lord36, Phoenix616
[18:50:23 WARN]: Plugin 'Pipes' is creating timing '## filter_amount' - this is deprecated behavior, please report it to the authors: Lord36, Phoenix616

What is happening?

While it's still working, the timings api is marked for removal and paper is nagging about it.
However, I belive while they make a fuss, they do not ship the api themselfs.

Additional context

You may want to have a look into spark docs
The Timings API is being used in io.github.apfelcreme.Pipes.Manager in lines 108, 241, 273, 320
where it's only ever called in form of try (Timing t = <Timing>.startTiming()) {...}

@Phoenix616
Copy link
Member

Until Spark can to what Timings v2 can do the hooks will not be removed from the plugin.

@FireInstall
Copy link
Author

Alright. Just wanted to open a issue for future refference

@FireInstall
Copy link
Author

Do you still stand by your opinion since timings is disabled by default now on paper? Does this mean everyone using Pipes has to re-enable them?

@Phoenix616
Copy link
Member

Phoenix616 commented Jul 23, 2024

Do you still stand by your opinion since timings is disabled by default now on paper?

Yes

Does this mean everyone using Pipes has to re-enable them?

No, only if you want Pipes-specific Timings.

Also if Paper ends up removing it completely I will look into adapting it to either Spark (if they ever manage to support this way of doing things), the inbuilt Minecraft profiler or even resurrecting my Paper fork Origami to re-add Timings v2.

@FireInstall
Copy link
Author

Well currently there is a PR on Paper to remove timings. I believe it won't get merged this update cycle but if mojang keeps introducing breaking updates in subversions maybe in some 1.21.X...

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

2 participants