Skip to content
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

Responsenet Implementation into SPRAS #176

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

gabeah
Copy link

@gabeah gabeah commented Jul 30, 2024

The wrapper implementation for ResponseNet.

Note: This is actively working, there are many issues that need to be ironed out. This is mainly for easier access for other maintainers to see the code.

@gabeah
Copy link
Author

gabeah commented Aug 12, 2024

Last update from me RE: ResponseNet wrapping.

As it stands, there is still an error with running the docker container. Anna and I were able to run interactively both my Docker image and the ReedCompBio image, but when trying to run it through SPRAS, the following error appeared:
image

Also as it stands, the wrapping is not yet complete. I started the wrapping process and its likely finished, but I was unable to do any testing of the wrapper due to previous errors. Thus, the things that need to be done are as follows:

  • Finish and test the wrapper functions for SPRAS
  • Add ResponseNet as a full fledged algorithm for SPRAS
  • Write Pytests

After creating this comment, I'm going to turn this PR into a draft PR for whenever it may be revived. I am also adding Anna, Tony, Altaf, and Oliver as contributors.

Have a good year y'all o7

@gabeah gabeah marked this pull request as ready for review August 12, 2024 23:34
@gabeah gabeah marked this pull request as draft August 12, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant