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

[Feature Request] Integration in Scipion tomo? #1

Open
pconesa opened this issue Jan 25, 2022 · 8 comments
Open

[Feature Request] Integration in Scipion tomo? #1

pconesa opened this issue Jan 25, 2022 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@pconesa
Copy link

pconesa commented Jan 25, 2022

Hi! Congratulations for this work.

I was wondering if integrating this into the scipion tomo ecosystem is something you would like to do.

Scipion can:

Do the installation, in this case creating a conda environment.
Smoothly integrate you software with the rest in the field.

Let me know, we can assist in the development process and codevelop the integration

@pconesa
Copy link
Author

pconesa commented Jan 27, 2022

HI again, in case you were not aware of what scipion is:

Scipion site: http://scipion.i2pc.es/

We recently organized a course on tomography. Content is deposited here:
Tutorials for tomography: https://scipion-em.github.io/docs/docs/user/user-documentation.html#tomography-tutorials

If you prefer videos, the course was recorded:
https://www.youtube.com/watch?v=RNlmYUepqWc&list=PLyJiuGnB9hAx8uy1g_4F4ypfOqvQjyCqU

All the best

@bbarad
Copy link
Contributor

bbarad commented Jan 28, 2022

So sorry for the delayed response - I've been underwater with other work the past few days. I'd be very very interested in integrating into scipion! I was super happy to see scipion integrate EMRinger (my first "polished" methods work), and I've been watching the pyseg integration with interest, and making our software more accessible (and making bookkeeping easier for users) would be a huge plus.

In terms of my immediate todo list, I want to:

  1. Set up a simplified conda.yml install file so that its a 1-step install. I assume this would also help with the scipion integration?
  2. Do some minor refactoring in the quantification code, which is currently a mix of general and reusable functions and mitochondria-specific routines that I'll need to get rid of before an integration is sensible.
  3. Write a detailed tutorial (ideally an interactive one!).

I think it might be appropriate to start talking about steps to integrate between 2. and 3. - do you agree?

@bbarad bbarad self-assigned this Jan 28, 2022
@bbarad bbarad added the enhancement New feature or request label Jan 28, 2022
@bbarad bbarad pinned this issue Jan 28, 2022
@pconesa
Copy link
Author

pconesa commented Jan 28, 2022 via email

@bbarad
Copy link
Contributor

bbarad commented Apr 2, 2022

I've made my environment.yml file - ended up being way more of a pain than I expected due to conflicting dependencies in some of the packages! Reference commit: 48c1e5f

@pconesa
Copy link
Author

pconesa commented Apr 3, 2022 via email

@bbarad
Copy link
Contributor

bbarad commented Apr 29, 2022

I've gotten v0.2 of the software out, with much more consistent pipelining for running the software. I think now may be a good time to come back around to scipion integration if you are still interested!

@pconesa
Copy link
Author

pconesa commented Apr 29, 2022

Sure, what about scheduling a meeting to discuss how to proceed and update each other on the status?

@bbarad
Copy link
Contributor

bbarad commented Apr 29, 2022

Sounds good - I am in GMT-7, so I am guessing morning meeting for me is the easiest overlap for you? I think any day but Wednesday next week would work for me!

@bbarad bbarad changed the title Integration in Scipion tomo? [Feature Request] Integration in Scipion tomo? Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants