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

Trees being cut off at mapblock boundries #5618

Closed
bigfoot547 opened this issue Apr 20, 2017 · 18 comments
Closed

Trees being cut off at mapblock boundries #5618

bigfoot547 opened this issue Apr 20, 2017 · 18 comments
Labels
Bug Issues that were confirmed to be a bug Duplicate Low priority @ Mapgen Non-trivial A large amount of work is required to address this (sometimes to the point of being infeasible)

Comments

@bigfoot547
Copy link
Contributor

screenshot_20170419_151122
Look at the Y coordinate.

@sfan5
Copy link
Collaborator

sfan5 commented Apr 20, 2017

related/dup: #5318

@MarkuBu
Copy link
Contributor

MarkuBu commented Apr 20, 2017

I don't think it is a duplicate. It is related, but a different issue.

@paramat paramat added @ Mapgen Bug Issues that were confirmed to be a bug labels Apr 20, 2017
@paramat
Copy link
Contributor

paramat commented Apr 20, 2017

Yes different issue.
y = 47 is a mapchunk border, i guess this is what you mean?
How often does this happen for you? It happens to me too but rarely, there is no obvious explanation.
Which mapgen? and are there any caves nearby?

@bigfoot547
Copy link
Contributor Author

bigfoot547 commented Apr 20, 2017 via email

@paramat
Copy link
Contributor

paramat commented Apr 20, 2017

I assume default trees, if these were custom trees > 16 nodes high they would be cut off at y = 63 instead.

@WSDguy2014
Copy link

WSDguy2014 commented Apr 21, 2017

Hmm, seems old glitch/bug is back, from 2010 or 2011. I'm not sure...

@Fixer-007
Copy link
Contributor

This thing is ultra rare for me, don't even remember when I last seen it.

@bigfoot547
Copy link
Contributor Author

@Fixer-007 That doesn't mean it needs to not be addressed

@paramat
Copy link
Contributor

paramat commented Apr 28, 2017

The rarity of this bug is variable for me, sometimes i see it 2-3 times in a session, sometimes don't see it for many days.

@paramat
Copy link
Contributor

paramat commented Sep 8, 2017

@bigfoot547 are these default MTGame apple trees?

@Wuzzy2
Copy link
Contributor

Wuzzy2 commented Sep 8, 2017

I recognize those textures. These are pine trees, maybe from v6.

@paramat
Copy link
Contributor

paramat commented Sep 8, 2017

Oh just seen 'pointing at pine needles' so yes.

@paramat
Copy link
Contributor

paramat commented Sep 6, 2018

I haven't seen this happen for a while now. maybe fixed somehow. Anyone else?

@Fixer-007
Copy link
Contributor

@paramat
Remember when I posted that screenshot with v7 river that had lots of trees near water been cut off from above? It was quite recently, somewhere in chat log with image...

@paramat
Copy link
Contributor

paramat commented Sep 9, 2018

Ok, thanks.

@paramat
Copy link
Contributor

paramat commented Aug 31, 2019

screenshot_20190831_020239

I recently did some testing of some other issue and set num_emerge_threads = 2 (default is 1). Then soon came across a large area of chopped trees at y = 47, the first time i had seen that for 1-2 years. I suspect a link, and it makes sense as these trees are passing beyond a mapchunk border, so are in an overlap volume.
The bug probably stopped happening for me when we set the default for num_emerge_threads to 1 a while back.

So it seems the cause is having > 1 emerge thread.

@paramat
Copy link
Contributor

paramat commented Nov 26, 2020

I realised this bug is probably be the same bug as the one documented in past versions of minetest.conf:

#    Number of emerge threads to use.
#    Make this field blank or 0, or increase this number to use multiple threads.
#    On multiprocessor systems, this will improve mapgen speed greatly at the cost
#    of slightly buggy caves.
#    type: int
# num_emerge_threads = 0

I seem to remember 'slightly buggy caves' meant the randomwalk caves being cutoff at mapchunk borders, creating flat walls. Just like trees, the randomwalk caves are structures that 'overgenerate': extend beyond a mapchunk border (dungeons do too).

So i suspect this warning in documentation was actually referring to all overgenerating structures occasionally being cutoff at mapchunk borders. If so, it includes this cutoff tree issue.

It might be a good idea to re-add a warning to current num_emerge_threads documentation about overgenerating structures (decorations, mid-sized caves, dungeons) occasionally getting cutoff at mapchunk borders when using multiple emerge threads.
I think the current documentation understates the multiple minor bugs caused by using multiple emerge threads.

Thinking about multiple emerge threads, it seems to make sense that problems might be caused with overgenerating structures.

In non-V6 mapgens, terrain overgenerates vertically by 1 node, so i suspect that issue #9357 is closely related to this.

@appgurueu
Copy link
Contributor

This is an instance of #9357 and should thus be closed as a duplicate.

@Zughy Zughy closed this as completed Jun 6, 2022
@Zughy Zughy added the Duplicate label Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that were confirmed to be a bug Duplicate Low priority @ Mapgen Non-trivial A large amount of work is required to address this (sometimes to the point of being infeasible)
Projects
None yet
Development

No branches or pull requests

9 participants