openscilab / pymilo

PyMilo: Python for ML I/O

MIT License

148 stars 5 forks source link

Anaconda Workflow #105

Closed sadrasabouri closed 5 months ago

sadrasabouri commented 6 months ago

Reference Issues/PRs

100

What does this implement/fix? Explain your changes.

Added

Anaconda workflow
Changed
Anaconda installation instruction added to README.md
Package's dependencies added to meta.yaml

Any other comments?

Do not merge this PR right before/after #104 due to probable silent merge conflicts.

AHReccese commented 5 months ago

@sadrasabouri LGTM, Thank you so much for your work. According to this issue it seems that the default access is write-all and sometimes(as discussed in the thread) in some projects suddenly this warning is getting raised and it is ok, even if you have not changed anything, so, I think it does address questions you raised in the last meeting, I tried to downgrade the workflow's access level from default write-all to read-all and it is now ok.

sepandhaghighi commented 5 months ago

@sadrasabouri, could you please rebase this PR and resolve any conflicts? Then ping me for the final review. 🔥

sadrasabouri commented 5 months ago

@sadrasabouri I think something went wrong during the rebase process. I can't see any changes in meta.yml.

meta.yml has been changed in last PR.