-
Notifications
You must be signed in to change notification settings - Fork 234
Closed
Labels
enhancementImproving an existing featureImproving an existing feature
Milestone
Description
A few thoughts on the design of loading the library as a whole (might be out of scope for this PR but worth mentioning):
- I initially made the
Sessionclass load the library when it's used to avoid doing this work at import time. Mostly so we wouldn't haveGMTLibNotFoundErrorat import. - That clearly didn't work because we need to call
beginat import for this whole thing to work. - So it might be time to rethink loading libgmt in the class instead of as a global instead to avoid searching for it every time.
- Or we figure out a way to not call
beginat import time.
Originally posted by @leouieda in #702 (comment)
Metadata
Metadata
Assignees
Labels
enhancementImproving an existing featureImproving an existing feature