dmipy
dmipy copied to clipboard
AthenaEPI
Dependencies
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b9415125811bb080b/dmipy/version.py#L60
@rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl .
I dunno. We should at some point reduce the amount of needed dependencies. Can we do something nice w/o seaborn?
Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d'Estienne d'Orves, 91120 Palaiseau
On 3 Feb 2018, at 13:03, Rutger Fick [email protected] wrote:
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
Not AS nice but scipy can do correlations and you can somehow make a plot showing it with matplotlib.
It just takes 7 hours to run the example (fit AxCaliber using mix) so it can be done for tomorrow. Is it a priority?
On 3 Feb 2018 13:07, "Demian Wassermann" [email protected] wrote:
I dunno. We should at some point reduce the amount of needed dependencies. Can we do something nice w/o seaborn?
Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Estienne d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Orves, 91120 Palaiseau https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g
On 3 Feb 2018, at 13:03, Rutger Fick [email protected] wrote:
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread <https://github.com/notifications/unsubscribe-auth/AI- 37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl> .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1#issuecomment-362801420, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37Hl2rnyQglxCcKVYpN5W-iQxylC7ks5tREvjgaJpZM4R4JMl .
Not a priority
-- Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d'Estienne d'Orves, 91120 Palaiseau
On 3 Feb 2018, at 13:09, Rutger Fick [email protected] wrote:
Not AS nice but scipy can do correlations and you can somehow make a plot showing it with matplotlib.
It just takes 7 hours to run the example (fit AxCaliber using mix) so it can be done for tomorrow. Is it a priority?
On 3 Feb 2018 13:07, "Demian Wassermann" [email protected] wrote:
I dunno. We should at some point reduce the amount of needed dependencies. Can we do something nice w/o seaborn?
Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Estienne d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Orves, 91120 Palaiseau https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g
On 3 Feb 2018, at 13:03, Rutger Fick [email protected] wrote:
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread <https://github.com/notifications/unsubscribe-auth/AI- 37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl> .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1#issuecomment-362801420, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37Hl2rnyQglxCcKVYpN5W-iQxylC7ks5tREvjgaJpZM4R4JMl .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
Also 7 hours to run a notebook example seems unreasonable. Maybe at add a warning on top of it?
-- Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d'Estienne d'Orves, 91120 Palaiseau
On 3 Feb 2018, at 13:09, Rutger Fick [email protected] wrote:
Not AS nice but scipy can do correlations and you can somehow make a plot showing it with matplotlib.
It just takes 7 hours to run the example (fit AxCaliber using mix) so it can be done for tomorrow. Is it a priority?
On 3 Feb 2018 13:07, "Demian Wassermann" [email protected] wrote:
I dunno. We should at some point reduce the amount of needed dependencies. Can we do something nice w/o seaborn?
Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Estienne d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Orves, 91120 Palaiseau https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g
On 3 Feb 2018, at 13:03, Rutger Fick [email protected] wrote:
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread <https://github.com/notifications/unsubscribe-auth/AI- 37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl> .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1#issuecomment-362801420, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37Hl2rnyQglxCcKVYpN5W-iQxylC7ks5tREvjgaJpZM4R4JMl .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
I'll change it over the next days then.
On 3 Feb 2018 13:11, "Demian Wassermann" [email protected] wrote:
Not a priority
-- Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France 1 Rue Honoré d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Estienne d https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g 'Orves, 91120 Palaiseau https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d'Orves,+91120+Palaiseau&entry=gmail&source=g
On 3 Feb 2018, at 13:09, Rutger Fick [email protected] wrote:
Not AS nice but scipy can do correlations and you can somehow make a plot showing it with matplotlib.
It just takes 7 hours to run the example (fit AxCaliber using mix) so it can be done for tomorrow. Is it a priority?
On 3 Feb 2018 13:07, "Demian Wassermann" [email protected] wrote:
I dunno. We should at some point reduce the amount of needed dependencies. Can we do something nice w/o seaborn?
Demian Wassermann, PhD [email protected] Associate Research Professor (CRN) Parietal Team INRIA Saclay Ile-de-France https://maps.google.com/?q=Ile-de-France+%0D+%3E+%3E+1+Rue+Honor%C3%A9&entry=gmail&source=g 1 Rue Honoré https://maps.google.com/?q=Ile-de-France+%0D+%3E+%3E+1+Rue+Honor%C3%A9&entry=gmail&source=g d <https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d' Orves,+91120+Palaiseau&entry=gmail&source=g> 'Estienne d <https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d' Orves,+91120+Palaiseau&entry=gmail&source=g> 'Orves, 91120 Palaiseau <https://maps.google.com/?q=1+Rue+Honor%C3%A9+d'Estienne+d' Orves,+91120+Palaiseau&entry=gmail&source=g>
On 3 Feb 2018, at 13:03, Rutger Fick [email protected] wrote:
So yes matplotlib for visualizations in the examples. Numba and pathos are really optional. I use seaborn for the last correlation plots in the AxCaliber example, should that be added too?
On 3 Feb 2018 12:55, "Demian Wassermann" [email protected] wrote:
https://github.com/AthenaEPI/dmipy/blob/ 3942f6e9c6b15372fd74ff6b941512 5811bb080b/dmipy/version.py#L60
@rutgerfick https://github.com/rutgerfick I think that the dependencies are too lax. We need to add everything that the tool needs to import to run. Boto is just needed for the examples so I added it but is there something missing? I think, for instance, that you are calling matplotlib from within the modules.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1, or mute the thread <https://github.com/notifications/unsubscribe-auth/AI- 37MMmxni1H7LxsgoSYhPKV-cuY2_xks5tREkygaJpZM4R4JMl> .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1#issuecomment-362801420, or mute the thread <https://github.com/notifications/unsubscribe- auth/AI-37Hl2rnyQglxCcKVYpN5W-iQxylC7ks5tREvjgaJpZM4R4JMl> .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AthenaEPI/dmipy/issues/1#issuecomment-362801656, or mute the thread https://github.com/notifications/unsubscribe-auth/AI-37Gg7bD8qvjXfjO8V9c7mZItnk5Zaks5tREzigaJpZM4R4JMl .
Added timing estimates for fitting data in the notebooks for every example. Will take care of the seaborn dependency later.