jolars / slopecd

4 stars 2 forks source link

Update illustration2d #49

Closed mathurinm closed 2 years ago

jolars commented 2 years ago

I added a bit of correlation between the columns in X, which usually helps, and tinkered with the other settings. It's possible to get something that's probably ever more effective, but I think this works as we want it to.

I modified the colors a bit too. I am going to work on coloring the line segments in the hybrid plot now.

image

jolars commented 2 years ago

I did a little more work on this and now have:

I think this is more or less what we want, since it shows the following:

image

Klopfe commented 2 years ago

I did a little more work on this and now have:

I think this is more or less what we want, since it shows the following:

  • The hybrid method can both move clusters in unison (merge) and split
  • Our method seems to converge faster
  • Our pure CD method cannot split clusters and eventually gets stuck

image

Very nice work! Do we wanna let the PGD go all the way to convergence or leave it as it is?

jolars commented 2 years ago

Very nice work! Do we wanna let the PGD go all the way to convergence or leave it as it is?

I'm not sure, what do you think? This way it's quite clear that PGD converges slower, but we could also just let PGD reach convergence and then mention how many steps it takes in the caption. Maybe that's better?

jolars commented 2 years ago

One thing I considered too was whether it would be nicer to have CD and Hybrid start with cd steps, i.e. follow exactly the same path until CD gets stuck, but that wouldn't match what we're doing exactly.

Klopfe commented 2 years ago

Very nice work! Do we wanna let the PGD go all the way to convergence or leave it as it is?

I'm not sure, what do you think? This way it's quite clear that PGD converges slower, but we could also just let PGD reach convergence and then mention how many steps it takes in the caption. Maybe that's better?

I'd prefer the second option but it's not so important.

Klopfe commented 2 years ago

One thing I considered too was whether it would be nicer to have CD and Hybrid start with cd steps, i.e. follow exactly the same path until CD gets stuck, but that wouldn't match what we're doing exactly.

Yes, I would rather keep it the way it is now. I think we clearly see the CD updates in only one direction and see that the first step is a PGD step. It shows what we do really well I think.

mathurinm commented 2 years ago

Very nice work! Do we wanna let the PGD go all the way to convergence or leave it as it is?

I'm not sure, what do you think? This way it's quite clear that PGD converges slower, but we could also just let PGD reach convergence and then mention how many steps it takes in the caption. Maybe that's better?

I think we need to let it go to convergence, since this will highlight that CD does not converge. Otherwise it may look like CD was not given enough iterations too

Maybe also puting all PGD updates in dashed (rightmost panel) would make it clearer why hybrid is "hybrid"?

And the figure is super neat, nice work!

jolars commented 2 years ago

Thanks for the input! Here's the "final" figure. Feel free to make edits if you feel like it.

image

mathurinm commented 2 years ago

It looks perfect! Thanks a lot. We see the huge steps, and the benefits of correction with PGD