Skip to content
This repository has been archived by the owner on Dec 20, 2021. It is now read-only.

Paper 1.14.x errors #32

Open
HauserGrim opened this issue Jun 30, 2019 · 22 comments
Open

Paper 1.14.x errors #32

HauserGrim opened this issue Jun 30, 2019 · 22 comments
Labels
accepted This bug is valid / This feature is gonna be added. bug Something isn't working

Comments

@HauserGrim
Copy link

Describe the bug
Console errors when join, using command and periodically

To Reproduce
Steps to reproduce the behavior:

  1. Run server with BVD and depencies
  2. Join to server
  3. Wait

Expected behavior
Work without errors

Timings
No need

Additional context
Paper 1.14.3 build 119
BVD dev build 35
Errors: https://pastebin.com/JFkb2PSk

And update dev builds link

@sofianedjerbi sofianedjerbi self-assigned this Jul 1, 2019
@sofianedjerbi sofianedjerbi added accepted This bug is valid / This feature is gonna be added. bug Something isn't working labels Jul 1, 2019
@sofianedjerbi sofianedjerbi removed their assignment Jul 1, 2019
@sofianedjerbi
Copy link
Owner

sofianedjerbi commented Jul 1, 2019

It seems that it comes from Paper. We'll need to wait for an update.

Z750 (Paper Developer) :

It comes from paper
API isn’t in 1.14.x and requires a deeper understanding of Mojangs rewrite of the chunk system to implement
"Okay, thanks. Are you planning to get this API compatible with 1.14 ?"
It’s planned
There are no ETAs

@Ristovski
Copy link

Maybe implement a more graceful way that detects Paper on 1.14.3 and prevents calling those APIs? This spams the server log quite a lot.

I am aware this is an issue from Paper itself, but seeing how this won't be implemented anytime soon (by the looks of it), perhaps a workaround is in order.

sofianedjerbi added a commit that referenced this issue Jul 12, 2019
@sofianedjerbi
Copy link
Owner

Maybe implement a more graceful way that detects Paper on 1.14.3 and prevents calling those APIs? This spams the server log quite a lot.

I am aware this is an issue from Paper itself, but seeing how this won't be implemented anytime soon (by the looks of it), perhaps a workaround is in order.

Added in commit f9cdc11

@Ristovski
Copy link

@Kugge Just to confirm, does this mean that BestViewDistance is broken on 1.14.3?

The commit disables it, I assume this is because it has no benefit without the per-player view distance APIs?

I thought it still set the global server view distance based on TPS/ping etc.

@sofianedjerbi
Copy link
Owner

sofianedjerbi commented Jul 13, 2019

@Ristovski The API provided by paper allow BVD to change the view distance, and that API is broken on 1.14.X. BVD cannot change view distance at all so it's 100% useless on 1.14.X

@Ristovski
Copy link

I see, I thought some of the functionality (like global view distance based on TPS/etc) was still there. Good to know.

@HauserGrim HauserGrim changed the title Paper 1.14.3 errors Paper 1.14.x errors Aug 4, 2019
@chistian95
Copy link

chistian95 commented Aug 31, 2019

Any update about this topic?

@Patbox
Copy link

Patbox commented Sep 23, 2019

Something changed?

@Cretezy
Copy link

Cretezy commented Sep 26, 2019

Where is this tracked upstream?

@SlickJava
Copy link

SlickJava commented Sep 30, 2019

the re-implementation of the patch has reached a good point where we can start external testing PaperMC/Paper#2572

this implements per player view distance at the ticket level, allowing more functionality and allowing shared view distance.

@SlickJava
Copy link

any testing would be much appreciated, that way we can get merged faster :))

@sisa1917
Copy link

sisa1917 commented Oct 2, 2019

Managed to get it working, seems good so far!
Here is my modified plugin https://sc.azureedge.net/BestViewDistance-12.1-MAJN.jar

Will report if anything breaks :)
Thank you @SlickJava

@SlickJava
Copy link

SlickJava commented Oct 4, 2019

just remember to keep the jar as updated as possible to the latest one on the PR (the latest one now built off of 209 with many fixes), other than that, happy testing

@nolsen42
Copy link

nolsen42 commented Oct 30, 2019

Running @SlickJava's patch + the latest paper version that I merged myself. The one huge problem I'm facing is my render distance is at 3, despite 20TPS. Not sure if this a bug, incompatibility, or misconfiguration.
image
image

@sisa1917
Copy link

Try running my patched version. This plugin will not work correctly in 1.14 without slight modifications.

@nolsen42
Copy link

@sisa1917 That's what I been running.
image

@sisa1917
Copy link

sisa1917 commented Oct 30, 2019

Interesting... Please add me on Discord Škýl#0001. I don't want to overly spam here (I will edit this comment with solution to this problem).

EDIT: figured out it was his Ping setting

@jewome62
Copy link

jewome62 commented Feb 6, 2020

Hello do you have new for update into 1.14 or 1.15 ?
This plugin is very usefull on my server
On paper the PR PaperMC/Paper#2572 was closed by author @SlickJava
So what's next?

@LoneDev6
Copy link

Hello do you have new for update into 1.14 or 1.15 ?
This plugin is very usefull on my server
On paper the PR PaperMC/Paper#2572 was closed by author @SlickJava
So what's next?

I'm curious too! :D

@Patbox
Copy link

Patbox commented Mar 10, 2020

PaperMC/Paper#2958

@jewome62
Copy link

jewome62 commented Apr 8, 2020

@Kugge Your plugin work with Tuinity 1.15.2
https://ci.codemc.io/job/Spottedleaf/job/Tuinity/

@TiredSysOp
Copy link

TiredSysOp commented May 2, 2020

@jewome62

@Kugge Your plugin work with Tuinity 1.15.2
https://ci.codemc.io/job/Spottedleaf/job/Tuinity/

Which build?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted This bug is valid / This feature is gonna be added. bug Something isn't working
Projects
None yet
Development

No branches or pull requests