-
Notifications
You must be signed in to change notification settings - Fork 2
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
Integrate within PYPIT #26
Comments
Ahhh... I had no idea that this was a plan. I suppose it makes sense. It depends if you think it has some value outside of PYPIT. Do you have a preference? |
Good point. It should have value outside PYPIT. Closing |
Reopening this issue... If we make changes to arclines, it seems unreasonable to expect that people using PYPIT will know to update both packages. It seems sensible, therefore, to include the arclines code into PYPIT. What do you think, @profxj? I wouldn't object to including the arc line lists too, for the record. |
I now agree. While I am going to merge #30 Maybe as a sub-package, if that is easy enough to |
i.e. no longer as a separate Repo.
What do you think @rcooke-ast ?
Is it time now?
The text was updated successfully, but these errors were encountered: