-
Notifications
You must be signed in to change notification settings - Fork 84
Dev Updates - 0.2.3 #636
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
Dev Updates - 0.2.3 #636
Conversation
Co-authored-by: Dario Coscia <[email protected]>
Co-authored-by: dario-coscia <[email protected]>
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! |
Co-authored-by: dario-coscia <[email protected]>
Co-authored-by: Federico Pichi <[email protected]>
Co-authored-by: dario-coscia <[email protected]>
* 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 <[email protected]> Co-authored-by: giovanni <[email protected]>
|
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 <[email protected]> Co-authored-by: Nicola Demo <[email protected]>
Description
This PR fixes:
#585
#594
#600
#619
#623
#624
#626
#630
#632
#644
#647
#649
Checklist