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

FastICA -- introduce coarse_limit to trigger switching to fine_g before reaching target limit #4

Merged
merged 4 commits into from Nov 21, 2012

Conversation

yarikoptic
Copy link
Contributor

per discussion on the mailing list about patched MDP in AFNI

Prantik Kundu and others added 4 commits November 20, 2012 11:39
Introduced to make things a bit clearer since there is already "fine tuning"
stage where optimization switches to used_g upon reaching the limit.
Calling the limit "coarse" instead of "primary/secondary" would help to preserve
original messages/logic thus not confusing existing users, while still allowing
to kick in gFine before reaching the limit
@yarikoptic
Copy link
Contributor Author

ah -- I have not spotted originally that there is first the generic parameters docs duplicated from parent classes ;)

BTW -- why not to make use of travis-ci.org -- comes for free ;) look e.g. at our
http://github.com/PyMVPA/PyMVPA/blob/HEAD/.travis.yml

@otizonaizit
Copy link
Member

ah -- I have not spotted originally that there is first the generic parameters docs duplicated from parent classes ;)

BTW -- why not to make use of travis-ci.org -- comes for free ;) look e.g. at our
http://github.com/PyMVPA/PyMVPA/blob/HEAD/.travis.yml

we are working on it already ;) stay tuned!

@yarikoptic
Copy link
Contributor Author

From "working on it" for any extended period of time my head would have imploded ;-) or is there some interesting complication?

Tiziano Zito notifications@github.com wrote:

ah -- I have not spotted originally that there is first the generic
parameters docs duplicated from parent classes ;)

BTW -- why not to make use of travis-ci.org -- comes for free ;)
look e.g. at our
http://github.com/PyMVPA/PyMVPA/blob/HEAD/.travis.yml

we are working on it already ;) stay tuned!


Reply to this email directly or view it on GitHub:
#4 (comment)

Sent from a phone which beats iPhone.

@Debilski
Copy link
Member

Am 20.11.2012 um 23:31 schrieb Yaroslav Halchenko notifications@github.com:

From "working on it" for any extended period of time my head would have imploded ;-) or is there some interesting complication?

Not really, we just weren’t smart enough to figure out that Erlang trick to simply use apt and get rid off those lengthy pip installations. ;)

/rike

Tiziano Zito notifications@github.com wrote:

ah -- I have not spotted originally that there is first the generic
parameters docs duplicated from parent classes ;)

BTW -- why not to make use of travis-ci.org -- comes for free ;)
look e.g. at our
http://github.com/PyMVPA/PyMVPA/blob/HEAD/.travis.yml

we are working on it already ;) stay tuned!


Reply to this email directly or view it on GitHub:
#4 (comment)

Sent from a phone which beats iPhone.

Reply to this email directly or view it on GitHub.

@yarikoptic
Copy link
Contributor Author

Ah, yeah, a nice truck from Matthew Brett IIRC

Rike-Benjamin Schuppner notifications@github.com wrote:

Am 20.11.2012 um 23:31 schrieb Yaroslav Halchenko
notifications@github.com:

From "working on it" for any extended period of time my head would
have imploded ;-) or is there some interesting complication?

Not really, we just weren’t smart enough to figure out that Erlang
trick to simply use apt and get rid off those lengthy pip
installations. ;)

/rike

Tiziano Zito notifications@github.com wrote:

ah -- I have not spotted originally that there is first the
generic
parameters docs duplicated from parent classes ;)

BTW -- why not to make use of travis-ci.org -- comes for free ;)
look e.g. at our
http://github.com/PyMVPA/PyMVPA/blob/HEAD/.travis.yml

we are working on it already ;) stay tuned!


Reply to this email directly or view it on GitHub:

#4 (comment)

Sent from a phone which beats iPhone.

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub:
#4 (comment)

Sent from a phone which beats iPhone.

otizonaizit pushed a commit that referenced this pull request Nov 21, 2012
FastICA -- introduce coarse_limit to trigger switching to fine_g before reaching target limit. 

Thanks to Prantik Kundu and Yaroslav Halchenko for the patch!
@otizonaizit otizonaizit merged commit 7bbd889 into mdp-toolkit:master Nov 21, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants