You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "orionlibs (via GitHub)" <gi...@apache.org> on 2023/07/18 15:41:28 UTC

[GitHub] [commons-math] orionlibs opened a new pull request, #234: MATH: created new clustering module and moved several classes from le…

orionlibs opened a new pull request, #234:
URL: https://github.com/apache/commons-math/pull/234

   initial commit for modularising the legacy clustering package. Created new clustering module and moved several classes from legacy into it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [commons-math] orionlibs commented on pull request #234: MATH-1579: created new clustering module

Posted by "orionlibs (via GitHub)" <gi...@apache.org>.
orionlibs commented on PR #234:
URL: https://github.com/apache/commons-math/pull/234#issuecomment-1641087227

   fixed checkstyle errors


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] MATH-1579: created new clustering module [commons-math]

Posted by "orionlibs (via GitHub)" <gi...@apache.org>.
orionlibs closed pull request #234: MATH-1579: created new clustering module
URL: https://github.com/apache/commons-math/pull/234


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [commons-math] aherbert commented on pull request #234: MATH-1579: created new clustering module

Posted by "aherbert (via GitHub)" <gi...@apache.org>.
aherbert commented on PR #234:
URL: https://github.com/apache/commons-math/pull/234#issuecomment-1641059233

   You should add `commons-math-clustering` to the top level pom.xml so it is included in the build reactor, then add it as a dependency to the `commons-math-legacy` module so this module can see the classes that you have moved.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [commons-math] orionlibs commented on pull request #234: MATH-1579: created new clustering module

Posted by "orionlibs (via GitHub)" <gi...@apache.org>.
orionlibs commented on PR #234:
URL: https://github.com/apache/commons-math/pull/234#issuecomment-1641067515

   @aherbert fixed


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [commons-math] orionlibs commented on pull request #234: MATH-1579: created new clustering module

Posted by "orionlibs (via GitHub)" <gi...@apache.org>.
orionlibs commented on PR #234:
URL: https://github.com/apache/commons-math/pull/234#issuecomment-1640736809

   Removed unused dependency


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [commons-math] orionlibs commented on pull request #234: MATH-1579: created new clustering module

Posted by "orionlibs (via GitHub)" <gi...@apache.org>.
orionlibs commented on PR #234:
URL: https://github.com/apache/commons-math/pull/234#issuecomment-1640645268

   The build failed because i declared in the pom.xml the artifact commons-math-clustering which doesn't exist yet in the repo. I didn't think of that. What do I have to do about it?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org