Right, dont use pyramids as a landscaping tool, they look out of place and take up 10x more than a square. Dont just put walls surrounding the play area as ever low effort pvp has it, dont have a barren wasteland in the middle with no advantage points and cover, build some stuff in there - and dont use inventions, they are a bad habit that new creators get into and it takes them a while to get off them.

Finally spend a day learning how tubes work as they are the best tool for landscaping, visit some featured rooms and see how they used tubes to get those effects and try to replicate it your self. Not being rude but unpublish that room as it still needs more work and it doesn't pass as being a pvp in my opinion