Kaushik Amar Das
Kaushik Amar Das
@cmarmo I have fixed the conflicts with the other tests. Please let me know if I need to make any other changes.
@glemaitre I implemented the changes as suggested.
> Arff I did not see that it was a merge conflict. @cozek Could you solve it I think I resolved the conflict.
@glemaitre @thomasjpfan Thank you for the reviews and discussions. Please tell me what changes I need to make regarding this PR.
@thomasjpfan @glemaitre Should I proceed with the changes as asked in https://github.com/scikit-learn/scikit-learn/pull/18555#issuecomment-886059885 ?
@glemaitre @thomasjpfan ping!
@thomasjpfan I hope I made the changes accordingly.
@thomasjpfan ping 😄
> Sorry, this PR dropped out of my radar during the 1.3 release. After we move the changes to 1.4, I am happy with merging this PR. No worries. I...
I can corroborate this. I am also getting the following error when using OpenFL: `ImportError: cannot import name 'DATASTORES' from 'metaflow.datastore'` metaflow 2.7 seems to be working it is unclear...