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

Add all MT_game trees to the mapgen. #2

Merged
merged 1 commit into from
Apr 7, 2016
Merged

Add all MT_game trees to the mapgen. #2

merged 1 commit into from
Apr 7, 2016

Conversation

mgl512
Copy link
Member

@mgl512 mgl512 commented Apr 7, 2016

Je crois que c'est la façon la plus efficace de générer tous les arbres de minetest_game : les enregistrer comme "décorations" v6. Je préfère faire une pull request que modifier directement le dépot parce que ce n'est pas une correction de bug.

Les valeurs sont arbitraires sauf la densité des arbres ("scale") et leur altitude. J'ai mis les aspens partout, les acacias près du niveau de la mer et les pins en altitude. Sans ces modifications, seuls des pins peuvent exister dans des biomes de neige je pense. C'est testé en local et je pense que tu voudras tester ça aussi en singleplayer.

En espérant que minetest_game ajoute ces arbres dans la mapgen v6 un jour pour éviter de la maintenance.

They are registered as schematic "decorations". The files without roots are used.
@paly2
Copy link
Member

paly2 commented Apr 7, 2016

Ok, je vais essayer de tester ce soir, éventuellement j'ajusterai les valeurs, mais je te fais confiance et je pense que je pourrai sans problème merger tel quel. Merci beaucoup pour cette très bonne idée !

@paly2 paly2 merged commit b8b5421 into MT-Eurythmia:master Apr 7, 2016
@paly2
Copy link
Member

paly2 commented Apr 7, 2016

C'est tout à fait excellent, je merge.
Mais c'est bizarre, une génération d'arbre devrait déjà être présente... https://github.com/mgl512/Mynetest-server/blob/trees/games/minetest_game/mods/default/mapgen.lua#L1142

@mgl512
Copy link
Member Author

mgl512 commented Apr 7, 2016

Mais c'est bizarre, une génération d'arbre devrait déjà être présente

Ce sont des "décorations" pour la mapgen v7, avec ses "biomes" qui n'existent pas dans la v6. La fonction qui les contient n'est pas appelée si la mapgen est v6, voir en fin de fichier.

@mgl512 mgl512 deleted the trees branch April 7, 2016 21:39
@paly2
Copy link
Member

paly2 commented Apr 8, 2016

Ah, ok.

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

Successfully merging this pull request may close these issues.

2 participants