Re: [eigen] Implementation/Integration of new linear solvers

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]

Hi Jens,

Without being able to give a definitive answer let me try to address some of your points:

a) Do you want the same level of coverage as for the already implemented solvers or more?
Are you talking about test coverage here? If yes you should probably try to match the coverage of existing solvers. Keep in mind that some tests exist to cover bugs fixed after the solvers got developed.

b) Would they typically go into the unsupported branch first?
First of all the dev branch contains supported and unsupported modules. They are just separated by the directory structure. If your patch extends supported parts of Eigen and if it fits the existing structure it would usually be merged into the supported part (ofc after reviewing it). However, if your patch extends an unsupported module (e.g. BlockSparseMatrix) it would probably be merged into the unsupported part.

c) Where should we discuss this? Should we create an issue/PR on Gitlab/the mailing list?
If you have general design questions the mailing list would be the appropriate place to discuss. If you have a ready or mostly ready implementation a Gitlab PR is the way to go. 

Additionally as a future project we want to work on BlockSparse Matrices. Eigen has some sort of implementation of it in 
Do you know of other people working on this and do you have any information on the implementation in the link?
Unfortunately I cannot give an answer here but I'm sure somebody will soon.

Cheers,
David


On 4. May 2020, at 09:23, Jens Wehner <J.Wehner@xxxxxxxxxxxxxxxxx> wrote:

Hello, 

anyone home? I posted this two weeks ago.

"
Hello Eigen-Developers,

We have been using Eigen for years now and we are very happy with the package. 

In the last year we have implemented two more linear solvers,BiGStab(l) and IDRstab, we would like to add them to Eigen. 

We are unsure about a couple of things:

a) Do you want the same level of coverage as for the already implemented solvers or more?

b) Would they typically go into the unsupported branch first?

c) Where should we discuss this? Should we create an issue/PR on Gitlab/the mailing list?

Additionally as a future project we want to work on BlockSparse Matrices. Eigen has some sort of implementation of it in 
Do you know of other people working on this and do you have any information on the implementation in the link?

Greetings 

Jens Wehner

| Jens Wehner, PhD | eScience Research Engineer | Email: j.wehner@xxxxxxxxxxxxxxxxx | Tel: +31(0)6 20 367 166 |
| Netherlands eScience Center | Science Park 140 | 1098 XG Amsterdam | The Netherlands |


From: Jens Wehner
Sent: Friday, April 17, 2020 11:55
To: eigen@xxxxxxxxxxxxxxxxxxx <eigen@xxxxxxxxxxxxxxxxxxx>
Subject: Implementation/Integration of new linear solvers
 
Hello Eigen-Developers,

We have been using Eigen for years now and we are very happy with the package. 

In the last year we have implemented two more linear solvers,BiGStab(l) and IDRstab, we would like to add them to Eigen. 

We are unsure about a couple of things:

a) Do you want the same level of coverage as for the already implemented solvers or more?

b) Would they typically go into the unsupported branch first?

c) Where should we discuss this? Should we create an issue/PR on Gitlab/the mailing list?

Additionally as a future project we want to work on BlockSparse Matrices. Eigen has some sort of implementation of it in 
Do you know of other people working on this and do you have any information on the implementation in the link?

Have a nice weekend and thanks for all your work

Jens Wehner 

| Jens Wehner, PhD | eScience Research Engineer | Email: j.wehner@xxxxxxxxxxxxxxxxx | Tel: +31(0)6 20 367 166 |
| Netherlands eScience Center | Science Park 140 | 1098 XG Amsterdam | The Netherlands |


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/