-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add editTopo.py tool #46
Conversation
Is this the latest version from here? https://github.com/COSIMA/topogtools/blob/6b4983127aa18dfdf1b62b2e18b581d82d4a64d4/editTopo.py |
Yes, I think so. |
41161f1
to
3917c90
Compare
If so, it might be good to include the commit hash or equivalently the URL https://github.com/COSIMA/topogtools/blob/6b4983127aa18dfdf1b62b2e18b581d82d4a64d4/editTopo.py in the git log to help with version control, since this script is now in 3 places and I'm not sure which of them should be thought of as definitive if we make improvements. |
3917c90
to
4d41387
Compare
@aekiss Good point! I've just added the url to the commit log. I would suggest we treat the different versions in the following way:
The same goes for all the other tools I think. Does this sound reasonable? |
Yes that makes sense. It will be tricky to avoid divergence between the versions if we make changes, but it would be nice to keep them synchronised. |
My thinking was that we wouldn't need to update the tools used for OM2 very often, if at all, because AFAIK, there are no planned updates to the OM2 topography. |
Fair point. So the version here should be though of as definitive/latest. |
We could put a note to that effect in the topogtools README |
Yes, I think that's a good idea. I now realize I should have discussed this in more detail with you at some point, so that we would be on the same page. Sorry about that. |
no problem! |
A couple more suggestions, but other than that it's ready to merge if the only thing you changed in the code was the GPL notice at the start (ie so I don't need to check the code). |
Co-authored-by: Andrew Kiss <[email protected]>
7d94d35
to
2cdc491
Compare
Yes, I've only added the copyright notice, everything else should be unchanged. Thanks for the review! |
No description provided.