-
Notifications
You must be signed in to change notification settings - Fork 185
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
GSLIB plotting functions should allow for axis specification #29
Comments
Howdy Jake,
That is a great idea. While ‘1-liners’ are useful for convenient, concise workflow code, flexibility is also important. It is a challenge to balance this! GeostatsPy was written on the weekends, days before students needed the code in my lectures! You will see many opportunities to improve.
o. I like this idea.
Remember, any improvements implemented in GeostatsPy will result in you being added to the author list with specification of your contribution!
All the best,
Michael
MICHAEL J. PYRCZ, Ph.D., P.Eng., Associate Professor
The University of Texas at Austin | Twitter<https://twitter.com/GeostatsGuy> | YouTube<https://www.youtube.com/channel/UCLqEr-xV-ceHdXXXrTId5ig> | GitHub<https://github.com/GeostatsGuy> | LinkedIn<https://www.linkedin.com/in/michael-pyrcz-61a648a1/> | Webpage<http://www.michaelpyrcz.com/> | Book<https://www.amazon.com/Geostatistical-Reservoir-Modeling-Michael-Pyrcz/dp/0199731446> | GoogleScholar<https://scholar.google.com/citations?user=QVZ20eQAAAAJ&hl=en> | DiReCT Consortium<https://direct.pge.utexas.edu/>
From: Jake Gearon <[email protected]>
Sent: Tuesday, September 15, 2020 4:27 PM
To: GeostatsGuy/GeostatsPy <[email protected]>
Cc: Subscribed <[email protected]>
Subject: [GeostatsGuy/GeostatsPy] GSLIB plotting functions should allow for axis specification (#29)
Hi Dr. Pyrcz,
I'm currently doing your HW #2<#2> and thought it might be useful to include axis arguments in locmap_st and hist_st so that one can supply ax1, ax2, ax3 args for further subplot customization.
I can try and create a PR sometime, but wanted to bring it up.
Jake G.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<#29>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AHUO3465NMESRGZA5NZY5R3SF7LZBANCNFSM4RNZTCAA>.
This message is from an external sender. Learn more about why this matters.<https://ut.service-now.com/sp?id=kb_article&number=KB0011401>
|
Pyrcz, Sounds good, I just try my best to flag things as I use open source packages (and hopefully PR them as a follow-up when possible). I'll do the PR this weekend. See you tomorrow! Jake |
keeping it open until I do it. Cheers |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi Dr. Pyrcz,
I'm currently doing your HW #2 and thought it might be useful to include axis arguments in locmap_st and hist_st so that one can supply ax1, ax2, ax3 args for further subplot customization.
I can try and create a PR sometime, but wanted to bring it up.
Jake G.
The text was updated successfully, but these errors were encountered: