-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fixing package entry point #111
base: main
Are you sure you want to change the base?
Conversation
thanks @Ciheim! How was this tested? Please document it stepwise here with an example. thanks! |
@Ciheim what is the expected behavior? I did a local install, but didn't see gen_intake_gfdl in my path by default. I am missing something though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation of tests in the PR are needed. My tests are not successful. Comments are on the PR for reference.
I'm going to have to go back over this. Seeing different behavior now. |
Should be working now. I had to change the method being used by the entry point so that it could understand the CLI arguments. Test: conda activate catalogbuilder gives: |
@aradhakrishnanGFDL When you have a chance please try it again and let me know how it goes. |
Removes the need to manually set path when creating a catalog.