PCRC icon indicating copy to clipboard operation
PCRC copied to clipboard

Support authlib-injector and display all part of skin

Open Sciroccogti opened this issue 5 years ago • 5 comments

感谢 @Merrrg1n

Sciroccogti avatar May 12 '20 09:05 Sciroccogti

Are you able to maintain this part of code if pycraft upgrades and changes related codes in the future?

Fallen-Breath avatar May 16 '20 10:05 Fallen-Breath

The part of ClientSettingsPacket code is hard-coded, which is not good in my opinion. Things like client view distance or something will somehow affect the recording

Fallen-Breath avatar May 16 '20 10:05 Fallen-Breath

Are you able to maintain this part of code if pycraft upgrades and changes related codes in the future?

Sadly I'm not sure, but I'll try, so maybe you'd better not to merge into master. I just hope this branch may help those who are using PCRC with authlib-injector.

Sciroccogti avatar May 18 '20 03:05 Sciroccogti

The part of ClientSettingsPacket code is hard-coded, which is not good in my opinion. Things like client view distance or something will somehow affect the recording

Maybe it can be configured with conf.json? Currently I haven't seen any codes in PCRC to configure it, so I guess these parameters are set as default now. However I didn't find out which is the default value of view distance.

I'll try to make them editable as long as I have time.

Sciroccogti avatar May 18 '20 04:05 Sciroccogti

Test is REQUIRED before merging!

I tried to do some easy things on reading conf.json, but do not have a chance to test it since our server is paused ~~(due to Graduate Entrance Exam)~~. And I also cannot decide which setting should be added into the conf.json.

Sciroccogti avatar Dec 05 '20 14:12 Sciroccogti