Dev updates#557
Conversation
* Add self-loop option to RadiusGraph and KNNGraph
* Improve grad logic and fix issues * Add operators' fast versions * Fix bug in laplacian + new tests + restructuring Co-authored-by: Dario Coscia <dariocos99@gmail.com> * fix advection bug --------- Co-authored-by: Dario Coscia <dariocos99@gmail.com>
* Refactoring solvers * Simplify logic compile * Improve and update doc * Create SupervisedSolverInterface * Specialize SupervisedSolver and ReducedOrderModelSolver * Create EnsembleSolverInterface + EnsembleSupervisedSolver * Create tests ensemble solvers * formatter * codacy * fix issues + speedup test
rm py3.8, add 3.13
* update tutorials * tutorial guidelines * doc
Code Coverage SummaryResults for commit: d0d8577 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
|
All tests passed, except for Codacy due to #541, the coverage it is still up to standards |
GiovanniCanali
left a comment
There was a problem hiding this comment.
Hi @dario-coscia!
Wouldn’t it be cleaner to simply rebase masteronto dev instead of merging? This way, we preserve the dev branch and avoid reviewing changes that have already been approved. Since everything in dev has gone through review, a rebase seems like the more straightforward option.
|
@GiovanniCanali I will do a rebase, but we still need 2 approvals since master is protected |
|
Somehow the tutorial workflow is failing, I will fix this before the rebase https://github.com/mathLab/PINA/actions/runs/14620520131 |
No description provided.