Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Satish Balay wrote: > On Fri, 10 Nov 2017, Satish Balay wrote: > $ git fetch -p && comm -12 <(git branch -r --merged origin/next-tmp | sort) > <(git branch -r --no-merged origin/master | sort) > origin/hongzh/add-tstraj-filename merged to master > origin/hongzh/copy_l2

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Matthew Knepley
On Fri, Nov 10, 2017 at 4:50 PM, Satish Balay wrote: > > On Fri, 10 Nov 2017, Richard Tran Mills wrote: > > > Hi Satish, > > > > Thanks for taking the initiative to switch to testing next-tmp to help > > clear up the constipation with moving things into master. It looks like > > there hasn't been

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: >> origin/jed/variadic-malloc > > http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2017/11/11/filtered-make_next-tmp_arch-mswin-uni_ps4.log > C:\cygwin64\home\petsc\PETSC~3.CLO\src\vec\is\utils\vsectionis.c(2380): > warning C4090: 'function': different 'const' qualifie

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Matthew Knepley writes: >> Alternative is to delete/recreate next - if needed. [but it requires >> all next users to do this delete/recreation] >> >> In the long term - Barry wants to get rid of next.. > > > 1) I think next really prevents master from getting screwed up (witness > next) Agree.

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Matthew Knepley
On Fri, Nov 10, 2017 at 8:56 AM, Jed Brown wrote: > Vaclav Hapla writes: > > >> 10. 11. 2017 v 5:09, Smith, Barry F. : > >> > >> > >> > >>> On Nov 8, 2017, at 3:52 AM, Vaclav Hapla > wrote: > >>> > >>> > 8. 11. 2017 v 9:06, Lisandro Dalcin : > > On 8 November 2017 at 05:51, Smith

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Matthew Knepley wrote: > > In the long term - Barry wants to get rid of next.. > > > 1) I think next really prevents master from getting screwed up (witness > next) > > 2) I think we are actually finding interaction bugs there. > > Are those points wrong, or is there anoth

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Matthew Knepley
On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay wrote: > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > In the long term - Barry wants to get rid of next.. > > > > > > 1) I think next really prevents master from getting screwed up (witness > > next) > > > > 2) I think we are actually finding

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Matthew Knepley writes: > On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay wrote: > >> On Sat, 11 Nov 2017, Matthew Knepley wrote: >> >> > > In the long term - Barry wants to get rid of next.. >> > >> > >> > 1) I think next really prevents master from getting screwed up (witness >> > next) >> > >>

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Matthew Knepley wrote: > On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay wrote: > > > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > > > In the long term - Barry wants to get rid of next.. > > > > > > > > > 1) I think next really prevents master from getting screwed up (

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Matthew Knepley
On Sat, Nov 11, 2017 at 12:53 PM, Satish Balay wrote: > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay > wrote: > > > > > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > > > > > In the long term - Barry wants to get rid of next.. > > > > > >

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > > I don't think we have the resources to run full tests on every branch one > > at a time. Do we? > > No, Well the hope is - after the migration to new test suite is complete the cost of a full test run is lower. And we could somehow do fewer tests to capt

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Jed Brown
Matthew Knepley writes: >> Matrix and graph are equivalent concepts. > > > This is clearly wrong. A matrix is the coordinate representation of a > linear operator, and thus has a specific > behavior under coordinate transformations. A graph is just connectivity, > and really just a relation. I ca

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Matthew Knepley
On Sat, Nov 11, 2017 at 1:12 PM, Jed Brown wrote: > Matthew Knepley writes: > > >> Matrix and graph are equivalent concepts. > > > > > > This is clearly wrong. A matrix is the coordinate representation of a > > linear operator, and thus has a specific > > behavior under coordinate transformation

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > BTW: Ultimlately if you want to improve current next model - everyone > > has to do a 'make alltests DIFF=$PETSC_DIR/bin/petscdiff' for atleast > > one build that has relavent feature options enabled - before merging > > the feature branch to nex

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> > I don't think we have the resources to run full tests on every branch one >> > at a time. Do we? >> >> No, > > Well the hope is - after the migration to new test suite is complete > the cost of a full test run is lower. And we c

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Matthew Knepley
On Sat, Nov 11, 2017 at 1:15 PM, Satish Balay wrote: > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > > > > BTW: Ultimlately if you want to improve current next model - everyone > > > has to do a 'make alltests DIFF=$PETSC_DIR/bin/petscdiff' for atleast > > > one build that has relavent feat

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > Satish Balay writes: > > > On Sat, 11 Nov 2017, Jed Brown wrote: > > > >> > I don't think we have the resources to run full tests on every branch one > >> > at a time. Do we? > >> > >> No, > > > > Well the hope is - after the migration to new test suite i

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Jed Brown
Matthew Knepley writes: > On Sat, Nov 11, 2017 at 1:12 PM, Jed Brown wrote: > >> Matthew Knepley writes: >> >> >> Matrix and graph are equivalent concepts. >> > >> > >> > This is clearly wrong. A matrix is the coordinate representation of a >> > linear operator, and thus has a specific >> > beh

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Matthew Knepley wrote: > And that makes my point. The time for me to login to cg, set > everything up, and run the tests should be automated, and in fact we > already did that for next, which is what should be used. You can automate an ssh command run a test on cg - [if you

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Matthew Knepley writes: > On Sat, Nov 11, 2017 at 1:15 PM, Satish Balay wrote: > >> On Sat, 11 Nov 2017, Matthew Knepley wrote: >> >> > > >> > > BTW: Ultimlately if you want to improve current next model - everyone >> > > has to do a 'make alltests DIFF=$PETSC_DIR/bin/petscdiff' for atleast >> >

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Matthew Knepley
On Sat, Nov 11, 2017 at 1:33 PM, Jed Brown wrote: > Matthew Knepley writes: > > > On Sat, Nov 11, 2017 at 1:12 PM, Jed Brown wrote: > > > >> Matthew Knepley writes: > >> > >> >> Matrix and graph are equivalent concepts. > >> > > >> > > >> > This is clearly wrong. A matrix is the coordinate rep

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> Satish Balay writes: >> >> > On Sat, 11 Nov 2017, Jed Brown wrote: >> > >> >> > I don't think we have the resources to run full tests on every branch >> >> > one >> >> > at a time. Do we? >> >> >> >> No, >> > >> > Well the hope

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > > Merging is synchronous. If I do > > git checkout master > git pull > git merge jed/risky-business > make alltests # works on my machine Note: this is my recommendation for the *currentK next model. If you are running 'make alltests' on your

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Matthew Knepley wrote: > >> And that makes my point. The time for me to login to cg, set >> everything up, and run the tests should be automated, and in fact we >> already did that for next, which is what should be used. > > You can automate an ssh com

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> >> Merging is synchronous. If I do >> >> git checkout master >> git pull >> git merge jed/risky-business >> make alltests # works on my machine > > Note: this is my recommendation for the *currentK next model. > > If y

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > > The way I see it - a broken next [where folks can't easily figure out > > who or which commit is responsible for the brakages] - doesn't help > > much.. > > The fundamental problem here is that we aren't accurate enough at > placing blame and getting the

Re: [petsc-dev] proposed minor PetscPartitioner changes

2017-11-11 Thread Jed Brown
Matthew Knepley writes: >> >> A graph Laplacian certainly does transform under coordinate >> >> transformation and indeed, we use that property to design effective >> >> coarsening strategies. That one basis strikes you as intrinsically >> >> "more canonical" does not mean it isn't a linear oper

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > > If you are running 'make alltests' on your laptop - then you don't need to > > test on es - before merge to maint. > > alltests takes hours and doesn't catch weird configurations -- you need > different PETSC_ARCH for that. It is normal to at least comp

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> > The way I see it - a broken next [where folks can't easily figure out >> > who or which commit is responsible for the brakages] - doesn't help >> > much.. >> >> The fundamental problem here is that we aren't accurate enough at >

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> > If you are running 'make alltests' on your laptop - then you don't need to >> > test on es - before merge to maint. >> >> alltests takes hours and doesn't catch weird configurations -- you need >> different PETSC_ARCH for that.

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > >> I think a lot of our noise in 'next' is "stupid shit", like compilation > >> failing on some architecture. Automating a very limited test suite > >> running on PRs within minutes should help a lot to deal with that. More > >> subtle interaction problems

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > The proposal I'm objecting to, and that has prevented me from writing an > important letter of recommendation this morning during some precious > time while Joule is sleeping, was to eliminate 'next'. We wouldn't have > needed to exchange dozens of emails i

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 11:17 AM, Matthew Knepley wrote: > > On Fri, Nov 10, 2017 at 4:50 PM, Satish Balay wrote: > > On Fri, 10 Nov 2017, Richard Tran Mills wrote: > > > Hi Satish, > > > > Thanks for taking the initiative to switch to testing next-tmp to help > > clear up the constipation wit

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 11:46 AM, Matthew Knepley wrote: > > On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay wrote: > On Sat, 11 Nov 2017, Matthew Knepley wrote: > > > > In the long term - Barry wants to get rid of next.. > > > > > > 1) I think next really prevents master from getting screwed up

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 11:33 AM, Jed Brown wrote: > > Matthew Knepley writes: > >>> Alternative is to delete/recreate next - if needed. [but it requires >>> all next users to do this delete/recreation] >>> >>> In the long term - Barry wants to get rid of next.. >> >> >> 1) I think next real

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
There is no reason to waste time protesting the attempt to change to the new model. The attempt will happen as soon as we have the new test harness fully working. So help out or get out of the way. Barry > On Nov 11, 2017, at 12:06 PM, Satish Balay wrote: > > On Sat, 11 Nov 2017, Jed B

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 11:49 AM, Jed Brown wrote: > > Matthew Knepley writes: > >> On Sat, Nov 11, 2017 at 12:37 PM, Satish Balay wrote: >> >>> On Sat, 11 Nov 2017, Matthew Knepley wrote: >>> > In the long term - Barry wants to get rid of next.. 1) I think next really p

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
Jed wrote (with my ()) I'm sure I'm not the only one in a similar situation. We need an effective set of tests that runs in less than five minutes (on all systems with all compilers) so that we can fix problems and move on rather than having lots of open threads hanging around. Sure, this

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
"Smith, Barry F." writes: >> On Nov 11, 2017, at 11:33 AM, Jed Brown wrote: >> >> Matthew Knepley writes: >> Alternative is to delete/recreate next - if needed. [but it requires all next users to do this delete/recreation] In the long term - Barry wants to get rid of next

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 3:20 PM, Jed Brown wrote: > > "Smith, Barry F." writes: > >>> On Nov 11, 2017, at 11:33 AM, Jed Brown wrote: >>> >>> Matthew Knepley writes: >>> > Alternative is to delete/recreate next - if needed. [but it requires > all next users to do this delete/recreati

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
Satish Balay writes: > On Sat, 11 Nov 2017, Jed Brown wrote: > >> >> I think a lot of our noise in 'next' is "stupid shit", like compilation >> >> failing on some architecture. Automating a very limited test suite >> >> running on PRs within minutes should help a lot to deal with that. More >>

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
"Smith, Barry F." writes: >There is no reason to waste time protesting the attempt to change >to the new model. The attempt will happen as soon as we have the >new test harness fully working. So help out or get out of the way. I can't help convert tests if I'm busy arguing against a

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > Removing next without a reliable substitute that ensures quality control > would be a disaster for the stability of 'master', and thus for everyone > trying to develop new features. That's what we had before switching to > Git and it was a mess. Sorry if I

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 3:26 PM, Jed Brown wrote: > > "Smith, Barry F." writes: > >> There is no reason to waste time protesting the attempt to change >> to the new model. The attempt will happen as soon as we have the >> new test harness fully working. So help out or get out of the way.

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
Let's please all stop wasting time arguing about next! We can argue when there is something to argue about. Not now. > On Nov 11, 2017, at 3:27 PM, Balay, Satish wrote: > > On Sat, 11 Nov 2017, Jed Brown wrote: > >> Removing next without a reliable substitute that ensures quality control

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
On Sat, 11 Nov 2017, Jed Brown wrote: > Satish Balay writes: > > > On Sat, 11 Nov 2017, Jed Brown wrote: > > > >> >> I think a lot of our noise in 'next' is "stupid shit", like compilation > >> >> failing on some architecture. Automating a very limited test suite > >> >> running on PRs within m

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
"Smith, Barry F." writes: > Jed wrote (with my ()) > > I'm sure I'm not the only one in a similar situation. We need an > effective set of tests that runs in less than five minutes (on all systems > with all compilers) so that we > can fix problems and move on rather than having lots of open

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Jed Brown
"Smith, Barry F." writes: >You are arguing against a change in the abstract because you love >next! You are making up stray men and attacking them. Wait until >there is a real proposal then point out flaws and make suggestions >on how to improve it. There is no reason to develop t

Re: [petsc-dev] broken nightlybuilds (next vs next-tmp)

2017-11-11 Thread Smith, Barry F.
> On Nov 11, 2017, at 3:49 PM, Jed Brown wrote: > > "Smith, Barry F." writes: > >> You are arguing against a change in the abstract because you love >> next! You are making up stray men and attacking them. Wait until >> there is a real proposal then point out flaws and make suggestions

[petsc-dev] nightlybuilds (next vs next-tmp)

2017-11-11 Thread Satish Balay
[Starting a new thread for this] Tonight's builds will be on 'next-tmp' with: $ git fetch -p && comm -12 <(git branch -r --merged origin/next-tmp | sort) <(git branch -r --no-merged origin/master | sort) |grep -v ' origin/next-tmp' origin/barry/add-bcaste-log origin/barry/fix-pastix-interfac