-
Notifications
You must be signed in to change notification settings - Fork 11
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
DM-22802: Changes requested by the DRAGONS team #76
base: main
Are you sure you want to change the base?
Conversation
@chris-simpson / @KathleenLabrie what is your current status regarding pex_config usage? Have you made additional changes since this PR was made? Have you stopped using it? |
We're still using a fork. I've make a few more changes I'd have to dig out. Some are associated with our command-line usage (for example, having ListField accept and parse a comma-separated string). |
b047819
to
a0f71e7
Compare
@ktlim I've had a go at rebasing this again. |
…ign more closely with DRAGONS logging levels
…than a list of such items
a0f71e7
to
9966481
Compare
This is a rebased version of #44 . It's slightly different than before because python 2 is no longer supported and the code was reformatted with black. The OrderedDict usage was dropped since dict is now ordered and I also protect the AstroData import.
Obvious things we need to think about:
__str__
can be used or if we have a method that a new type must support.cc/ @chris-simpson