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

Things to do before v1.9 #306

Closed
2 of 3 tasks
simonschoelly opened this issue Sep 20, 2023 · 4 comments
Closed
2 of 3 tasks

Things to do before v1.9 #306

simonschoelly opened this issue Sep 20, 2023 · 4 comments
Labels
question Further information is requested
Milestone

Comments

@simonschoelly
Copy link
Contributor

simonschoelly commented Sep 20, 2023

I looked a bit at the PRs that we have introduced since v1.8 - it looks mostly good, but there are a few things that we probably should fix before a next release, so that we don't introduce new bugs:

Minor:

  • Some normalized_cut() fixes #84 was merged, but that PR uses some magic constant - I am not completely sure that this is the right way - but on the other hand we could argue that this replaces a bug with another one, so maybe we can ignore it.
@gdalle gdalle added this to the v1.9 milestone Sep 21, 2023
@gdalle
Copy link
Member

gdalle commented Sep 21, 2023

Thanks for taking the time!

  1. Wanna do it, since you wrote all of the GenericGraph infrastructure?
  2. Yeah it has something to do with JET on nightly. My fix would be to keep nightly CI (and not Stop running CI on nightly? #295) but decouple it from the "build passing" badge
  3. Do you mean dnz = abs.(diag(D)) .>= 1E-16? Maybe we can do eps(T) instead?

@gdalle
Copy link
Member

gdalle commented Sep 21, 2023

What do you think about #307?

@gdalle gdalle added the question Further information is requested label Sep 21, 2023
@gdalle
Copy link
Member

gdalle commented Sep 21, 2023

Okay, it seems #307 can wait and so can the workaround from #84.

As far as I'm concerned, once #305 and #309 are merged, we're good to tag

@simonschoelly
Copy link
Contributor Author

@gdalle we can close this, right?

@gdalle gdalle closed this as completed Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants