cooljeanius / dawn_of_thunder

a Wesnoth UMC campaign; forked to clean up (originally by Paulomat4)
https://forums.wesnoth.org/viewtopic.php?f=8&t=39512
GNU General Public License v2.0
2 stars 1 forks source link

Update maps to meet knyght's criteria for map quality #8

Closed cooljeanius closed 9 months ago

cooljeanius commented 1 year ago

From wesnoth/wesnoth#7479, according to @knyghtmare, good maps should have:

  1. More than 10 terrain types used.
  2. Map is actually visually appealing to look at, i.e., not too plain (this one is still kind of subjective IMO)
  3. Map appearance uses the campaign's lore properly.
  4. Uses embellishments.
cooljeanius commented 1 year ago

So yeah @knyghtmare I rechecked the first map (for S01, "The Outpost") and it turns out it does actually meet your criteria: it uses 14 types of base terrain (Gs, Re, Gg, Hh, Rp, Cv, Kv, Ww, Wwf, Wwrg, Rd, Mm, Ce, Ke), plenty of embellishments, and fits the campaign lore of being the southern edge of a forest

knyghtmare commented 1 year ago

Map is actually visually appealing to look at, i.e., not too plain (this one is still kind of subjective IMO)

Still not impressed. Sorry. Also, the Wesnoth Modders Guild have an impressive roster of map-makers: I mean you can ask for improvements there.

cooljeanius commented 1 year ago

Map is actually visually appealing to look at, i.e., not too plain (this one is still kind of subjective IMO)

Still not impressed. Sorry. Also, the Wesnoth Modders Guild have an impressive roster of map-makers: I mean you can ask for improvements there.

I think it's perfectly fine and you're just being too picky

(edit: but I'm still going to keep this bug open until I've checked the rest of the maps, too, though)

knyghtmare commented 1 year ago

I mean whatever. it's your campaign. If you don't wanna update maps to higher quality, then don't

I am retiring soon anyways, so probably wont matter..

cooljeanius commented 1 year ago

hm, I wonder if there's actually a way to automate checking against these criteria? Opening each map in the editor and then counting the terrain types used by hand is going to be tedious...

cooljeanius commented 1 year ago

Discovered an error when trying to load the S03 map in the editor:

20230403 22:48:33 warning editor: format error in load map : Unknown tile in map: (Cv^Cvw) 'Cv^Cvw'
Screen Shot 2023-04-03 at 22 48 44

Any idea how to fix? The map loads successfully when playing normally in gameplay mode...

cooljeanius commented 1 year ago

ok I'll admit that the map for S07, "Deep Forest", could use some work

cooljeanius commented 1 year ago

addition of the "help wanted" label is for the S07 map; it's a bit tricky there since adjusting the map will require adjusting the scenario code, too, since there is some terrain-dependent code in it

cooljeanius commented 9 months ago

Actually nvm; c6f47b5 was probably enough.