Skip to content
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

Overlapping help in rungeneric.py and related files #39

Closed
GoogleCodeExporter opened this issue Aug 8, 2015 · 5 comments
Closed

Overlapping help in rungeneric.py and related files #39

GoogleCodeExporter opened this issue Aug 8, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

the help of rungeneric.py and rungenericmany.py etc. are very much overlapping 
(as one calls the other in case). Is there a better way to organize this? 

Original issue reported on code.google.com by [email protected] on 14 Jun 2014 at 11:42

@GoogleCodeExporter
Copy link
Author

Original comment by [email protected] on 16 Jun 2014 at 12:19

  • Added labels: Code-Postprocessing

@GoogleCodeExporter
Copy link
Author

Original comment by [email protected] on 12 Mar 2015 at 3:44

  • Changed state: Accepted

@brockho brockho self-assigned this Aug 10, 2015
@brockho
Copy link
Contributor

brockho commented Apr 29, 2017

In the long run, rungeneric1.py, rungeneric2.py, and rungenericmany.py should be integrated into the rungeneric.py such that the problem with the overlapping help will be solved along this refactoring. This will also help to remove a lot of duplicated code for reading in and checking data.

@nikohansen
Copy link
Contributor

See also #1364, quite related.

@brockho brockho added this to the cocopp2 milestone Nov 22, 2017
@brockho
Copy link
Contributor

brockho commented Apr 28, 2020

With the changes from end of March in the development branch, the overlapping docstrings of rungenericmany.py, rungeneric1.py, and rungeneric.py are cleaned up now. The rungenericmany.py and rungeneric1.py are only called from within rungeneric.py and thus accessed via a single entry point from both the command line and from cocopp.main(...).

@brockho brockho closed this as completed Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants