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

method_MULTILAYER #17

Open
naveedishaque opened this issue Dec 8, 2023 · 1 comment · May be fixed by #153
Open

method_MULTILAYER #17

naveedishaque opened this issue Dec 8, 2023 · 1 comment · May be fixed by #153
Assignees

Comments

@naveedishaque
Copy link
Contributor

naveedishaque commented Dec 8, 2023

Update metadata:
https://docs.google.com/spreadsheets/d/1QCeAF4yQG4bhZSGPQwwVBj_XF7ADY_2mK5xivAIfHsc/edit#gid=0

Details of method (publication, GitHub):
https://www.cell.com/cell-systems/fulltext/S2405-4712(21)00151-4
https://github.com/SysFate/MULTILAYER

@uniqueg
Copy link

uniqueg commented Dec 14, 2023

MULTILAYER's main functionality (the one we would want to benchmark) is GUI-based. Unfortunately, the GUI stuff is not just a thin shim that collects arguments and passes them to some class that we could easily trigger without the GUI. And all of the code is in a 3.6k line single module that will be hard to patch - while making sure we are retaining the intended functionality (at least I cannot do that with my zero knowledge of spatial transcriptomics). My recommendation is to either skip this method or ask the dev to create a CLI- or proper library-type entry point that we can hook in.

In any case, I will create a PR for the environment file I created (which was also a pain) - in case anyone feels up for a challenge 💪

@uniqueg uniqueg linked a pull request Dec 14, 2023 that will close this issue
@uniqueg uniqueg linked a pull request Dec 14, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants