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

program analysis also in delphi #14

Closed
cl4yton opened this issue May 20, 2018 · 3 comments
Closed

program analysis also in delphi #14

cl4yton opened this issue May 20, 2018 · 3 comments
Assignees
Labels
question Further information is requested

Comments

@cl4yton
Copy link
Contributor

cl4yton commented May 20, 2018

@adarshp : Posting as "question" for discussion, although I'm "stating" it here...
The program analysis project right now has the following components:
(1) Analyze fortran to map to python
(2) Analyze pythons AST to map to CAG with functions that can be input to delphi
(3) Sensitivity analysis of delphi CAG with functions
Item (3) will be in delphi (has general use).
For now, I'd like to put parts of (2) also in the delphi project, under the directory program_analysis/ (at project root, sibling to sensitivity). For now I'll keep this in the sensitivity branch. This means probably adding Jon Stephens to the project.
Long term: This may move out depending on whether we consider the python side of program analysis a component of delphi (which currently I'm ok with it)

@cl4yton cl4yton added the question Further information is requested label May 20, 2018
@adarshp
Copy link
Collaborator

adarshp commented May 20, 2018

I think this sounds like a good idea. We can keep program analysis-specific package dependencies as optional while they are in flux using the method here.

@cl4yton
Copy link
Contributor Author

cl4yton commented May 20, 2018

I didn't know setup tools could do that (I have used them much, but this is cool).
I think the program-analysis-specific work, focusing on translating from python.AST to export delphi_DBN json should be very lightweight -- it really should only need pure python.
Sensitivity, however, will include tangent (https://github.com/google/tangent) -- when I get to that.

@adarshp
Copy link
Collaborator

adarshp commented May 20, 2018

Awesome, less messing around with the arcana of setuptools = more happiness for me :)

@adarshp adarshp closed this as completed May 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants