Precompilation warning

[aelligp]

The latest commit introduced a "duplicated" function and now GMG doesn't get precompiled:

┌ GeophysicalModelGenerator
│  WARNING: Method definition extract_ProfileData!(GeophysicalModelGenerator.ProfileData) in module GeophysicalModelGenerator at /Users/pascalaellig/.julia/packages/GeophysicalModelGenerator/TBwKS/src/ProfileProcessing.jl:397 overwritten at /Users/pascalaellig/.julia/packages/GeophysicalModelGenerator/TBwKS/src/ProfileProcessing.jl:415.
│  ERROR: Method overwriting is not permitted during Module precompilation. Use `__precompile__(false)` to opt-out of precompilation.

The functions can be found here

[mthielma]

I guess this is somewhat similar to the issue I recently had with Interactive Geodynamics, which we did not resolve so far.
The idea of introducing that duplicate function was to allow older function calls without the ScreenshotData argument. I guess we could also move the ScreenshotData argument to the optional arguments to avoid defining a duplicate.

[albert-de-montserrat]

If I'm not wrong, when calling that function, only the second method can be called. So effectively, the first method can't be used. You should get rid of one of the two methods, if you want two separate methods you can do multiple dispatch tricks along this way:

foo(x; a=1, b=nothing) = foo(x, a, b)
foo(x, a, ::Nothing) = 1
foo(x, a, b) = 2