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

Remove vplants.tissue_analysis dependencies #19

Open
pradal opened this issue Feb 1, 2016 · 2 comments
Open

Remove vplants.tissue_analysis dependencies #19

pradal opened this issue Feb 1, 2016 · 2 comments

Comments

@pradal
Copy link
Contributor

pradal commented Feb 1, 2016

Several dependencies already exist to vplants.tissue_analysis.
Even if these deps are imported at run-time, the code is not fully functional today.
Several options exist to remove these dependencies:

  1. Inherit of the data structure in vplants.tissue_analysis and add the algos in the inherited class
  2. Seprarte the datastructure from the algos and use a service as a factory (I want to build my data-structure from images. This kind of solution is used in NetworkX for instance.
@pradal pradal changed the title Remove cplants.tissue_analysis dependencies Remove vplants.tissue_analysis dependencies Feb 1, 2016
@jlegrand62
Copy link
Contributor

Sorry, I did not see this issue! I'm addressing it right now in a branch "wip_tissue_analysis_dependencies".
I will first remove the dependencies and keep a maximum of functionalities there to avoid too many changes within the code.
In order to keep a working "TissueAnalysis" package, I have made another branch "wip_Tissue4D" that save the present state of container.

Jo

@jlegrand62
Copy link
Contributor

@gcerutti as-tu un test dans cellcomplex ou sam4Dmaps qui utilise un TemporalPropertyGraph ?

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

2 participants