search

OSIPI / TF2.4_IVIM-MRI_CodeCollection

OSIPI TF2.4: IVIM MRI code collection
Apache License 2.0
9 stars 27 forks source link

report on repo #56

Closed abhicodes369 closed 4 months ago

abhicodes369 commented 6 months ago

Describe the changes you have made in this PR

created a report on the repository like the name of the contributor and how many are being tested A clear and concise description of what you want to happen a simple HTML is being generated if you execute this script

Link this PR to an issue [optional]

Fixes #47

Checklist

abhicodes369 commented 5 months ago

I apologize I accidentally deleted the git ignore file can I create a new branch

etpeterson commented 5 months ago

I apologize I accidentally deleted the git ignore file can I create a new branch

Sure, start over or just add it back on this branch, either one.

etpeterson commented 5 months ago

This is looking good, I think you're almost there. After these small changes I'll send it around to the contributors for a final sanity check.

etpeterson commented 4 months ago

Just checking on this, are these changes still in the works?

abhicodes369 commented 4 months ago

image it looks like this. Are there any modifications required?

etpeterson commented 4 months ago

image it looks like this. Are there any modifications required?

I had 3 comments that look like they haven't been addressed yet. Small but useful like choosing empty rather than no.

abhicodes369 commented 4 months ago

i am sorry but i couldnt find these comments could you please give the link to comments that state necessary changes

etpeterson commented 4 months ago

i am sorry but i couldnt find these comments could you please give the link to comments that state necessary changes

Apologies, I didn't finish the review so my comments didn't show for you.

etpeterson commented 4 months ago

Looks like a merge conflict in requirements.txt.