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 new map "Old Ruin" #338

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Atlante1952
Copy link

@Atlante1952 Atlante1952 commented Jun 3, 2024

Hello, here is my map, which I made, I would like to integrate it into the the flag capture server on minetest; currently no error, I made a size neither too big nor too small, there are 2 camps (red and blue) The map is available for the three game modes (classic, classes, nade_fight) There are minerals for the classic in the structures that emerge from the earth. And we can't get off the map
screenshot

@Atlante1952 Atlante1952 changed the title Map "Old Ruin" Add new map "Old Ruin" Jun 4, 2024
@SoulS33ker
Copy link
Contributor

SoulS33ker commented Jun 6, 2024

PR FILES
1.conf ✔
2.mts ✔
3.screenshot 3:2 ✘
MAP
1.No errors ✔
2.functional treasure chests ✔
3.flag capturable ✔
4.team chests ✔
5.Indestructible flag area ✘
6.build time barriers ✘ (touching middle barrier teleports you)
screen-recording.webm vs screen-recording(2).webm

MECHANICS
1.equal chance of winning ✔
2.base pos ✔
3.terrain balance ✔
4.ore count ✘ (BLUE: 24 iron,7mese,5dia; RED:18iron,6mese,3dia )
5.roof limit ✘ (red roof is 1 layer less than blue's limit; artificial ignore/void nodes detected)

MISCELLANEOUS
1.floating tree & stone(on bridge) detected (ignorable)
screenshot_20240606_013332
2.random dead spaces detected (torches were placed by me)
screenshot_20240606_014623
RECOMMENDATIONS
1.make water source on bridge manually indestructible/unreachable +adding indestructible stones around it
screenshot_20240606_013255
OVERALL
a better version of river valley i would say,good map approve worthy with minor changes :)

@Themostrandom
Copy link
Contributor

Hello, I really like the look of your map, there are however some problems: cool water from the bridge, your map seems to be RP so maybe we should remove this water or add a tunnel filled with water in your bridge, SoulS33ker there are a lot of holes in the ground (I saw some he had not seen I think). On the edges, some of the holes/cliffs are unnecessary in my opinion (3 blocks on each side). The indestructible blocks under the spawn/flag are missing.

screenshot_20240614_144247
screenshot_20240614_144252
screenshot_20240614_144303
screenshot_20240614_144308
screenshot_20240614_144358

@HobbitPower
Copy link
Contributor

HobbitPower commented Aug 6, 2024

@Atlante1952

Beautifully built map. However, there are still a few things to consider:

  • Please note what SoulSeeker has already said. Both sides should have the same amount of ore. I would also like to see more ores for both sides, at least more iron.
  • Your team zones are 1 block too small. You get teleported back if you stand directly next to outer barriers or the center.
  • The 5x5 area of the flag must consist of indestructible blocks.
  • You used the wrong glass for the outer barriers: Use ctf_map:ind_glass not ctf_map:glass.
  • The screenshot must have a 3:2 ratio.
  • I would also suggest making the map a bit smaller, it is very open to the top.
  • Why do you use CC-BY-SA-3.0 instead of 4.0?
  • Not the most important thing but yeah, you could fix the "holes" underground and the floating trees.

  • The water can stay in my opinion, as well as the cliffs (see TMR's comment). A water tunnel would be also okay however. (Admittedly, I don't know what he means by "RP".)

@HobbitPower
Copy link
Contributor

RECOMMENDATIONS 1.make water source on bridge manually indestructible/unreachable +adding indestructible stones around it screenshot_20240606_013255

Would be better, but you can still block the water this way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants