search

geodynamics / burnman

BurnMan is a library for modeling mantle thermodynamics and thermoelasticity constrained by mineral physics experiments.
https://geodynamics.github.io/burnman/
GNU General Public License v2.0
53 stars 41 forks source link

New blurb at top of py files #124

Closed bobmyhill closed 8 years ago

bobmyhill commented 9 years ago

Can we decide on a new standard blurb at the top of each file? Something to include CIG, and add 2015? Also, egocentric, I know, but can we add me? :)

We should also change the Github link from burnman.org to the CIG page.

ian-r-rose commented 9 years ago

You are right on all counts.

tjhei commented 9 years ago

I vote for

# This file is part of BurnMan - a lower mantle toolkit.
# Copyright (C) 2012 - 2015 by the BurnMan team, released under the GNU GPL v2 or later.

(short)

We should also change the Github link from burnman.org to the CIG page.

Sure, can you make a PR?

bobmyhill commented 9 years ago

Hmmm, I would make a small modification:

# This file is part of BurnMan - a thermoelastic and thermodynamic toolkit for the Earth and Planetary Sciences.
# Copyright (C) 2012 - 2015 by the BurnMan team, released under the GNU GPL v2 or later.

We've moved a long way beyond the original brief for the project!

bobmyhill commented 9 years ago

As for the Github link (the one at the top of the main GH page), I don't think I have the appropriate permissions to change that...

tjhei commented 9 years ago

Oh, that one. I think we should keep burnman.org there and figure out a way to prominently link to the CIG page from there.

bobmyhill commented 9 years ago

Ok, I'm fine with that...

ian-r-rose commented 9 years ago

I think the toplevel text you have written is fine, Bob.

bobmyhill commented 9 years ago

Ok, great. I'll wait until we've resolved the other PRs to work on this, as I'd rather not make a mess.

sannecottaar commented 9 years ago

Very much approved.

On Fri, Mar 6, 2015 at 8:56 PM, Bob Myhill notifications@github.com wrote:

Ok, great. I'll wait until we've resolved the other PRs to work on this, as I'd rather not make a mess.

— Reply to this email directly or view it on GitHub https://github.com/geodynamics/burnman/issues/124#issuecomment-77624624.

CaymanUnterborn commented 9 years ago

Looks good to me.

On Sat, Mar 7, 2015 at 3:36 AM, Sanne Cottaar notifications@github.com wrote:

Very much approved.

On Fri, Mar 6, 2015 at 8:56 PM, Bob Myhill notifications@github.com wrote:

Ok, great. I'll wait until we've resolved the other PRs to work on this, as I'd rather not make a mess.

— Reply to this email directly or view it on GitHub <https://github.com/geodynamics/burnman/issues/124#issuecomment-77624624 .

— Reply to this email directly or view it on GitHub https://github.com/geodynamics/burnman/issues/124#issuecomment-77679625.