Conversation
Co-authored-by: Dario Coscia <dariocos99@gmail.com>
Code Coverage SummaryResults for commit: 8442969 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
|
Hi @dario-coscia @FilippoOlivo!
Most of them are almost ready, let's try to finish them all. |
Hi @GiovanniCanali, I commented some of them because I believe they will require a bit more time. Are you ok with this change? |
Sure, fine for me! |
* add normalizer callback * implement shift and scale parameters computation * change name files normalizer data callback * reduce tests * fix documentation * add NotImplementedError for PinaGraphDataset --------- Co-authored-by: FilippoOlivo <filippo@filippoolivo.com> Co-authored-by: giovanni <giovanni.canali98@yahoo.it>
|
I would add #602 and #639, and then merge. What do you think @dario-coscia @FilippoOlivo? |
* add maintainers * add codeowners * add governance * add affiliation Filippo Olivo in the committee/maintainers list * update affiliation for Nicola Demo in the committee/maintainers list * update governance policy to reflect PINA terminology * Clarify PINA Organisation references in charter (#651) * rm @mstrazzu from maintainers --------- Co-authored-by: FilippoOlivo <filippo@filippoolivo.com> Co-authored-by: Nicola Demo <demo.nicola@gmail.com>
Description
This PR fixes:
#585
#594
#600
#619
#623
#624
#626
#630
#632
#644
#647
#649
Checklist