DevoLearn / devolearn

Accelerate data driven research on embryos with deep learning models :microscope:
MIT License
43 stars 31 forks source link

dependency management and changes for main.yml file... #issue 81 and #issue71 #82

Closed sushumereddy closed 1 year ago

sushumereddy commented 1 year ago

here in this we are changing from setup.py system to requirements.in file ,we have updated required libraries in requirements.in file by seeing import files and when we run pip-compile requirements.in

requirements.txt file is generated,according to files mentioned in requirements.in and it is used for better dependency management.so further we don't need the file setup.py so we removed it

this following pr resolves the following issue 71 and issue81

the following issues are run in my forked repo even workflow was runned in my local environment ,here are screenshots of of the tests passed for 3.9 and 3.10.9 versions of python

Screenshot 2023-02-11 at 12 31 42 AM Screenshot 2023-02-11 at 12 31 53 AM Screenshot 2023-02-11 at 12 32 07 AM Screenshot 2023-02-10 at 11 45 55 PM Screenshot 2023-02-11 at 12 31 08 AM Screenshot 2023-02-11 at 12 31 31 AM

so ,once pls review my pr and help me knew if anything is needed

codecov-commenter commented 1 year ago

Codecov Report

Base: 86.22% // Head: 86.32% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (42eea2d) compared to base (8b90b20). Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #82 +/- ## ========================================== + Coverage 86.22% 86.32% +0.09% ========================================== Files 12 12 Lines 421 424 +3 ========================================== + Hits 363 366 +3 Misses 58 58 ``` | [Impacted Files](https://codecov.io/gh/DevoLearn/devolearn/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [...cell\_membrane\_segmentor/cell\_membrane\_segmentor.py](https://codecov.io/gh/DevoLearn/devolearn/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGV2b2xlYXJuL2NlbGxfbWVtYnJhbmVfc2VnbWVudG9yL2NlbGxfbWVtYnJhbmVfc2VnbWVudG9yLnB5) | `87.28% <0.00%> (+0.10%)` | :arrow_up: | | [...neage\_population\_model/lineage\_population\_model.py](https://codecov.io/gh/DevoLearn/devolearn/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGV2b2xlYXJuL2xpbmVhZ2VfcG9wdWxhdGlvbl9tb2RlbC9saW5lYWdlX3BvcHVsYXRpb25fbW9kZWwucHk=) | `81.73% <0.00%> (+0.17%)` | :arrow_up: | | [...n/embryo\_generator\_model/embryo\_generator\_model.py](https://codecov.io/gh/DevoLearn/devolearn/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGV2b2xlYXJuL2VtYnJ5b19nZW5lcmF0b3JfbW9kZWwvZW1icnlvX2dlbmVyYXRvcl9tb2RlbC5weQ==) | `84.61% <0.00%> (+0.24%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

sushumereddy commented 1 year ago

pls don't merge this pr there are some code changes should be added