DynamicSurroundingsFabric icon indicating copy to clipboard operation
DynamicSurroundingsFabric copied to clipboard

1.18.2 support and fixes

Open ThexXTURBOXx opened this issue 4 years ago • 33 comments

Fixes #13 Fixes #10

ThexXTURBOXx avatar Dec 20 '21 14:12 ThexXTURBOXx

Heh, you just decided to ignore the exception? Okay, that's a decision, too 😃

Well, until a proper fix is found, it doesn't hurt to ignore it. The channel is released anyways (it is getting released twice in fact which is the issue) ;)

ThexXTURBOXx avatar Dec 20 '21 14:12 ThexXTURBOXx

@ThexXTURBOXx no offence, but you could have just made a PR with the style fixes and then notify on my PR that I have to update to 1.18.1, no? PRs are usually meant for a single topic and shouldn't address multiple things imo. Not saying you should close this, I'm just leaving this as a friendly reminder :D I have updated dependencies on my PR now anyway.

spnda avatar Dec 21 '21 18:12 spnda

Wait, sorry, I apparently did not get a notification about spnda/DynamicSurroundingsFabric#1. I take everything I've said back, you probably just thought I was inactive :D

spnda avatar Dec 21 '21 18:12 spnda

@spnda Oh okay, no problem! Yeah, that's pretty annoying that GitHub doesn't notify you about PRs on forks by default. Yes, I thought you may have better things to do, sorry for that misunderstanding! :)

ThexXTURBOXx avatar Dec 21 '21 18:12 ThexXTURBOXx

@ThexXTURBOXx, I do not want to be obtrusive, but it seems https://github.com/ThexXTURBOXx/DynamicSurroundingsFabric/pull/2 PR solves the current problem with the crash of the game, during the game there was not a single crash.

HarvelsX avatar Dec 21 '21 19:12 HarvelsX

@HarvelsX Well, I didn't get a notification either :) Will look at it asap. Thank you very much!

ThexXTURBOXx avatar Dec 21 '21 19:12 ThexXTURBOXx

Yeah, that's pretty annoying that GitHub doesn't notify you about PRs on forks by default.

@ThexXTURBOXx that is a very weird default... Btw, want to cherry-pick the new version of my commit? I did some minor changes, aswell as bumped modmenu to 3.0.1 👍🏻

spnda avatar Dec 21 '21 19:12 spnda

@spnda Merged your changes instead of cherry picking them (in order to give full credit), thank you :)

@HarvelsX Same applies to your changes. I also fixed the workflow that you were working on and included it! Thank you very much, your fix seems to be working for me as well!

ThexXTURBOXx avatar Dec 21 '21 23:12 ThexXTURBOXx

The addition of the GitHub Actions workflow feels out of scope for a port

ChloeDawn avatar Dec 22 '21 21:12 ChloeDawn

in 1.16.4 there where fireflies flying around flowers and now in 1.18.1 fabric they are gone ?

Lumnati avatar Jan 10 '22 23:01 Lumnati

also the village sounds are gone why ?

Lumnati avatar Jan 10 '22 23:01 Lumnati

Hi this is probably not really useful otherwise to the conversation or bug issues but I've been trying to find where to download the branched 1.18.1 version of dynamic surroundings. It's a little confusing to find if it's even accessible or not here, since there is the 1.18 version on the other main pull (right?) that has the direct link.

Like I know how to do a download through the github action thing but beyond that could someone point me in the right direction? I know it's in beta or alpha or w/e, that's fine because I've had no issue really with the 1.18 version so far and I've just been wanting to update my game to 1.18.1 finally.

Thanks.

PlayfulPiano avatar Jan 15 '22 22:01 PlayfulPiano

@Lumnati Might have a look at this soon, thanks for noticing!

@pianoboy8 Clone my repo and build the jar yourself :)

ThexXTURBOXx avatar Jan 15 '22 22:01 ThexXTURBOXx

How do you build the jar?

PlayfulPiano avatar Jan 15 '22 23:01 PlayfulPiano

How do you build the jar?

Here's another way to get this great mod on 1.18.1 here https://github.com/HarvelsX/DynamicSurroundingsFabric/actions/runs/1641972118 These are the latest commits with PR

HarvelsX avatar Jan 16 '22 10:01 HarvelsX

@HarvelsX thank you!!

PlayfulPiano avatar Jan 16 '22 17:01 PlayfulPiano

How do you build the jar?

Here's another way to get this great mod on 1.18.1 here https://github.com/HarvelsX/DynamicSurroundingsFabric/actions/runs/1641972118 These are the latest commits with PR

There seems to be a problem with this version of raindrops? image

Dr-WeiAL avatar Jan 17 '22 18:01 Dr-WeiAL

My version can be downloaded here: https://github.com/Smokeybear12/DynamicSurroundingsFabric/actions/runs/1703503069 @Smokeybear12 forked my fork and added GitHub Actions. If you encounter the issue there as well, this might need further digging. However, I don't exactly see what's wrong in your picture.

ThexXTURBOXx avatar Jan 17 '22 18:01 ThexXTURBOXx

My version can be downloaded here: https://github.com/Smokeybear12/DynamicSurroundingsFabric/actions/runs/1703503069 @Smokeybear12 forked my fork and added GitHub Actions. If you encounter the issue there as well, this might need further digging. However, I don't exactly see what's wrong in your picture.

I tried your two versions, the problem is the same, the specific performance is that the ripples on the water surface are twitching and twitching, and there are no ripples on the water surface in some places

1 2 3

Dr-WeiAL avatar Jan 17 '22 19:01 Dr-WeiAL

My version can be downloaded here: https://github.com/Smokeybear12/DynamicSurroundingsFabric/actions/runs/1703503069 @Smokeybear12 forked my fork and added GitHub Actions. If you encounter the issue there as well, this might need further digging. However, I don't exactly see what's wrong in your picture.

link not work

Skidamek avatar Jan 18 '22 18:01 Skidamek

link not work

You seem to be too late. He removed his fork. Just use the HarvelsX's verison instead.

ThexXTURBOXx avatar Jan 18 '22 18:01 ThexXTURBOXx

There seems to be a problem with this version of raindrops? image

@Dr-WeiAL The bug seems to have been fixed, try to use it: https://github.com/HarvelsX/DynamicSurroundingsFabric/suites/5151891727/artifacts/155652131

HarvelsX avatar Feb 02 '22 14:02 HarvelsX

There seems to be a problem with this version of raindrops? image

@Dr-WeiAL The bug seems to have been fixed, try to use it: https://github.com/HarvelsX/DynamicSurroundingsFabric/suites/5151891727/artifacts/155652131

Yes, the ripples in this version are no longer messy, and it looks much more comfortable.

Dr-WeiAL avatar Feb 02 '22 15:02 Dr-WeiAL

@Dr-WeiAL, it's good that it helped you, but next time create a separate issue (https://github.com/OreCruncher/DynamicSurroundingsFabric/issues) on something.

HarvelsX avatar Feb 02 '22 15:02 HarvelsX

Does this build have any issues with 1.18.2?

Sunconure11 avatar Mar 02 '22 18:03 Sunconure11

Does this build have any issues with 1.18.2?

I made a commit adding support for 1.18.2, but be careful, it's untested. I would be glad to have your help in testing. https://github.com/HarvelsX/DynamicSurroundingsFabric/tree/1.18

HarvelsX avatar Mar 04 '22 16:03 HarvelsX

Might it at all be possible to get in contact w/orecruncher and see if it is at all possible to upload the recent 1.18/1.18.1/eventual 1.18.2 versions on like curseforge or modrinth so people have easier access to the files (even if they're more in an alpha/beta status?

PlayfulPiano avatar Mar 07 '22 07:03 PlayfulPiano

Let's phrase it like this: Anyone is permitted to upload it by himself since this mod is under MIT License. So, if someone wants to upload it, go ahead! :)

ThexXTURBOXx avatar Mar 07 '22 09:03 ThexXTURBOXx

Would it be possible to put compiled jars in here every so often, for testing purposes?

Sunconure11 avatar Mar 07 '22 09:03 Sunconure11

They are available here: https://github.com/ThexXTURBOXx/DynamicSurroundingsFabric/actions If you click on the latest build and download the artifact :)

ThexXTURBOXx avatar Mar 07 '22 09:03 ThexXTURBOXx