Skip to content

Add "Big Ocean" map by gameit#48

Merged
ClobberXD merged 8 commits into
MT-CTF:masterfrom
JostP:big_ocean
Oct 29, 2019
Merged

Add "Big Ocean" map by gameit#48
ClobberXD merged 8 commits into
MT-CTF:masterfrom
JostP:big_ocean

Conversation

@JostP
Copy link
Copy Markdown
Member

@JostP JostP commented Aug 27, 2019

see #35

Screenshot is missing

@JostP JostP mentioned this pull request Aug 27, 2019
@ClobberXD ClobberXD changed the title Add "big ocean" map by gameit Add "Big Ocean" map by gameit Aug 27, 2019
@Thomas--S
Copy link
Copy Markdown
Member

Please add license information.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Sep 4, 2019

Please add license information.

and screenshot (400 x 600 image (for /maps command))

ClobberXD changed the title Add "big ocean" map by gameit Add "Big Ocean" map by gameit 8 days ago

I don't have time to occupy myself with spelling on school holidays :)

@ClobberXD
Copy link
Copy Markdown
Contributor

I don't have time to occupy myself with spelling on school holidays :)

No worries, I exist for a reason ;)

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Sep 30, 2019

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Sep 30, 2019

We still need screenshot

@ClobberXD
Copy link
Copy Markdown
Contributor

👍 for everything except for the inconsistency of using destructible nodes throughout the map but (unintentionally?) making the sails out of indestructible white wool. Will approve once that's fixed.

@JostP Here's a screenshot, which you can use, if you think it does justice to the map:

screenshot_20191002_152731

@Thomas--S
Copy link
Copy Markdown
Member

Please be aware that license information still seems to be missing.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 3, 2019

Well, there is also second (quite big) problem: Caves and ores are only on blue side.

Screenshot after replacing stone with air:
Zaslonska slika 2019-10-03 15-51-37

@ClobberXD
Copy link
Copy Markdown
Contributor

Should those be removed altogether?

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 4, 2019

Quote gameit:

Big Ocean 2.0
After a time of thinking, I finished the new version of my map "Big Ocean".

  • Changed the deep of the water from 10 to 20
  • And placed ctf-stone in the middle of the map, so you can't build a tunnel from one base to the other
  • Added a new bigger type of island
  • Put some minerals/caves in the stone
  • Added a skypart
  • Placed more ships

Ores maybe should stay if player get stone sword and not steel sword

BTW Thank you for adding me to map reviewers team :D

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 6, 2019

So what do you think, should ores stay or be removed? If they stay, should player get stone sword?

@ClobberXD
Copy link
Copy Markdown
Contributor

If gameit can add ores to the other side, then I'd personally prefer that approach. If not, I'd prefer to not have any ores, and give steel swords to players.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 20, 2019

Update:

Map now have:

  • more than 300 000 iron
  • more than 300 000 coal
  • more than 300 000 copper 😄

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The map doesn't have any license. Also if the players spawn in with steel swords then there is no need for iron ores. If the players are to be spawned in with stone swords then also the amount of ores is too much. The player should also spawn in with some torches. Rest seems fine and if need custom chest zones can be added later on. The .conf file also needs requires the screenshot line(screenshot = big_ocean.png) to be added.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 20, 2019

What about if ores aren't removed? You get steel sword, so you can have extra steel to craft doors, traps etc. Wouldn't that be great, to have hundreds of doors in-game, bases built with rainforced stone or damage-cobble (both are not possible to craft many on normal maps)?

@ghost
Copy link
Copy Markdown

ghost commented Oct 23, 2019

Yes it does provide for various items like traps and doors to be made but I highly doubt it would be used for that since crafting(crafting of traps and doors) is only used in Pro Vs Pro matches.

@ghost
Copy link
Copy Markdown

ghost commented Oct 24, 2019

Players when they get steel swords usually don't tend to go for mining of resources like iron because their utility of steel sword is already fulfilled so they don't need to go for mining. Ice Age is a good example here, the map has I think 99+ iron ores yet players don't go mining and they like to engage directly in combat as there is no incentive to go and mine.

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 24, 2019

Just for information, what means 300 000+ blocks of each ore :)
Zaslonska slika 2019-10-24 10-09-42

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 28, 2019

Added license version

@ghost ghost self-requested a review October 28, 2019 11:01
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably give players some torches as initial items? I don't mind having that much abundance of iron. Rest everything looks fine. Will approve it once torches are given as initial items. (I was thinking whether Harbour would have been a better name for this map)

@Thomas--S
Copy link
Copy Markdown
Member

To be honest, I'm not sure if we really should allow GPL licenses for maps. They are intended for source code, but maps are media files, so I guess that there could be problems when applying GPL to maps. (What is the "source code" of a map?)

Please note: I am not a lawyer.

I strongly suggest not to merge this map until all open questions regarding licensing are solved.

@ClobberXD
Copy link
Copy Markdown
Contributor

To be honest, I'm not sure if we really should allow GPL licenses for maps. They are intended for source code, but maps are media files, so I guess that there could be problems when applying GPL to maps. (What is the "source code" of a map?)

Disclaimer: IANAL

"Source code" refers to the actual data that makes up the licensed work. In this context, the schematic file is the source code.

https://www.gnu.org/licenses/gpl-faq.html#GPLOtherThanSoftware

I strongly suggest not to merge this map until all open questions regarding licensing are solved.

Apart from whether using GNU GPL v3.0 is a good choice for non-software works, the only legal-ish issue that prevents this PR from being merged:

This, however, makes me wonder if it's wrong to just display the name. I think we should allow map makers to add the license name, and a file containing the full fat license deed.

#48 (comment)

@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 28, 2019

Added torch as initial stuff

@ghost
Copy link
Copy Markdown

ghost commented Oct 28, 2019

Yeah I think the source code here is the schematic file since it is the "main" thing the map creator created.
Quoting from the GNU site,

You can apply the GPL to any kind of work, as long as it is clear what constitutes the “source code” for the work. The GPL defines this as the preferred form of the work for making changes in it.

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@ghost ghost added the One approval ✅ label Oct 28, 2019
@JostP
Copy link
Copy Markdown
Member Author

JostP commented Oct 28, 2019

Maybe we should still ask gameit about license version before merging

@ghost
Copy link
Copy Markdown

ghost commented Oct 28, 2019

Maybe we should still ask gameit about license version before merging

I did ask them and they themselves weren't sure but said that it should be version 3, so the PR is good to go.

@ClobberXD
Copy link
Copy Markdown
Contributor

ClobberXD commented Oct 28, 2019

@JostP Did you add 300000 of each ore, or was it gameit?

Nvm

@Thomas--S
Copy link
Copy Markdown
Member

The GPL defines this [the source code] as the preferred form of the work for making changes in it.

I am not sure if the schematics are really "source code" with this definition. At least I prefer to make the changes to the actual Minetest world (which is not available) rather than directly modifying schematics.

@ClobberXD
Copy link
Copy Markdown
Contributor

I am not sure if the schematics are really "source code" with this definition. At least I prefer to make the changes to the actual Minetest world

But you publish the changes via the schematic itself, right? It's like saying that you don't modify the code directly, but you only modify the copy of the file that your text editor has loaded. :)

(Disclaimer: IANAL)

@Thomas--S
Copy link
Copy Markdown
Member

As always, I am not a lawyer.

But to me it seems like distributing only the schematic is like distributing the compiled program. It is theoretically possible to make changes to the machine code (i.e. schematic) directly, but most developers prefer to make the changes to the source code (here the Minetest world).

@ClobberXD
Copy link
Copy Markdown
Contributor

it seems like distributing only the schematic is like distributing the compiled program

Unlike a compiled program in machine code, a schematic is just a serialised version of a Lua table, which can be de-serialised either manually, or by using minetest.read_schematic (5.1.0+ only, IIRC).

@ghost
Copy link
Copy Markdown

ghost commented Oct 29, 2019

Copy link
Copy Markdown
Contributor

@ClobberXD ClobberXD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@ClobberXD ClobberXD merged commit 72e754c into MT-CTF:master Oct 29, 2019
@Thomas--S
Copy link
Copy Markdown
Member

From MinetestSam's link:

Think of 'source' as something which is created in a manual process and then fed into an automated process to produce the final artifacts.

As far as I can understand, the Minetest world is shaped in a manual process and then fed into an automated process (i.e. map exporter) to produce the final artifacts. (i.e. schematics).


In future, please don't merge maps as long as staff members have worries regarding the licensing. Having to wait a few days longer until open questions are discussed is a lot less of a problem than having legal problems regarding licensing.

I'd strongly suggest to revert this merge commit.

Please note: I am not a lawyer.

ghost pushed a commit that referenced this pull request Oct 29, 2019
@ghost ghost mentioned this pull request Oct 29, 2019
ClobberXD pushed a commit that referenced this pull request Oct 29, 2019
ghost pushed a commit that referenced this pull request Oct 29, 2019
@ClobberXD
Copy link
Copy Markdown
Contributor

ClobberXD commented Oct 29, 2019

As far as I can understand, the Minetest world is shaped in a manual process and then fed into an automated process (i.e. map exporter) to produce the final artifacts. (i.e. schematics).

Am I correct in assuming that you seem to be agreeing to the use of GPL for this map?

Taken from https://opensource.stackexchange.com/a/742:

The source code for a work means the preferred form of the work for making modifications to it.
What's the preferred form for making modifications? You might deduce what your project's source code is by going through the following sequence:

  • Consider the work that is supplied to the recipient.

  • Consider a range of possible changes that recipient might want to make: move images around, give a character different movements in an animated movie, alter text, change the order of sections in a document, bring obscured background elements to the foreground unobstructed, change a single instrumental track in an audio file, etc.

  • What format of the work would make those editing tasks easiest?

Usually, the answer will be obvious: it's the layered Photoshop or GIMP file you used to create your finished image, or the word processing document you use to create your PDF output, or the multi-track audio-workshop file you compressed down into only a two-band stereo output. For a web page with images, the source code would be non-obfuscated HTML and the most-easily editable versions of the images.

Sometimes (I'd guess rarely) the answer is not obvious. For example, the Battle for Wesnoth community has had a non-trivial discussion about whether MIDI or OGG constitutes the preferred form for making modifications, because MIDI is closer to editable "source" but exactly how a MIDI will render to a proper sound file will vary by MIDI environment. Aside from strange edge cases like this, it should be obvious what document format is most appropriate for making modifications.

Other noteworthy cases:

  • If the work has always been in a form that is not friendly to making modifications (e.g., a complex image made in a simple Paint-style program without layer information, or a binary executable written by hand by a very clever hacker with a hex editor) then that is the best "source" that exists and is sufficient: that was, apparently, the original author's preferred format for creating and modifying the work.
  • Conversely, if the project has always been in a format most conducive to making modifications (e.g., human-readable HTML for a website), then the work is composed entirely of source code.
  • For the simple-image-in-Photoshop hypothetical, if the Photoshop file has metadata what would make it easier for a person to perform edits, that should be preserved: e.g., if you specified gradients/regions for color alterations and that information is available as separately-editable layer-like feature, then certainly that would be preferred for making modifications. If the transformation you did merely altered pixels and has no associated metadata or layer information, then the Photoshop file affords no advantage over the raw image in terms of ease of modification. However, unless I were absolutely sure that no such editing-friendly metadata existed, I would err of the side of caution and distribute the PSD file as well.

GPL certainly isn't the best license for non-software works, but it should work just fine as far as I can see. Or, of course, we can also ask the author to reconsider their choice of license for the map.

Disclaimer: IANAL

@ghost ghost mentioned this pull request Nov 1, 2019
@JostP JostP deleted the big_ocean branch November 23, 2019 12:50
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.

3 participants