-
Notifications
You must be signed in to change notification settings - Fork 13
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 example_obj_lib.py file using objects from tutorial.ipynb and use in main script #151
Comments
Re-opening because, although #155 created the example_obj_lib.py file, I haven't yet updated the notebook (which I will rename from example.ipynb to tuotiral.ipynb) to use this package. |
(edited for clarity) Slight hiccup: the examples section of the docs, which gets compiled with $CWD being This is because Another option: this could be avoided if we explicitly included the example library in aospy: @spencerkclark any thoughts? Hope I'm being clear enough. |
@spencerahill are you referring to this page? http://aospy.readthedocs.io/en/latest/examples.html |
Are you trying to add an example of using the main script there (which I assume would benefit from using |
Could creating a symbolic link to
That feels kind of hacky though. What do you think? |
Yes, or at least Nice idea re: symbolic link. Another option is interactively adding the examples directory to PYTHONPATH via |
The least hack-ish option to me is to actually add We could also include |
I agree; I think doing that would be perfectly fine.
Yeah, I say hold off on adding anything else until we explicitly need it. |
@spencerkclark perfect, going this route. |
@spencerahill also there are a couple spots in the tutorial notebook that need to be changed now that it's been moved to a new directory. I can create a pull request tomorrow if you'd like. |
@spencerkclark I noticed some also. I'll have a PR in by the end of the night; feel free to push anything that I missed once it's up. |
Essentially just copy and paste the object definitions in examples.ipynb into a separate .py file that is itself a functional aospy object library. That way users have a direct example of a working object library file from the outset that they can copy elsewhere and manipulate, rather than having to cut and paste from a notebook.
Also, we should populate the main script with this object library, such that users can submit the main script and execute example calculations immediately upon installing aospy.
The text was updated successfully, but these errors were encountered: