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

Documentation for starfiles #470

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

jojoelfe
Copy link
Contributor

@jojoelfe jojoelfe commented Apr 1, 2024

This should add the starfile node to the documentation. I am still working on the tutorial, but figured I open the PR to see if the CI for the docs works correctly.

ALso included are some small fixes for relion import that I noticed while making a tutorial.

Copy link

codecov bot commented Apr 1, 2024

Codecov Report

Attention: Patch coverage is 37.50000% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 77.66%. Comparing base (6e82364) to head (5f6b596).
Report is 6 commits behind head on main.

❗ Current head 5f6b596 differs from pull request most recent head 748997e. Consider uploading reports for the commit 748997e to get more accurate results

Files Patch % Lines
molecularnodes/io/parse/star.py 37.50% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #470      +/-   ##
==========================================
- Coverage   77.74%   77.66%   -0.08%     
==========================================
  Files          40       40              
  Lines        3612     3617       +5     
==========================================
+ Hits         2808     2809       +1     
- Misses        804      808       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BradyAJohnston
Copy link
Owner

The docs build OK on my machine - I need to look into why they aren't auto-triggering a build for the PR.

@BradyAJohnston
Copy link
Owner

BradyAJohnston commented Apr 2, 2024

Now changed so that the docs will build & deploy on every PR :)

EDIT:
OK I remember why I disabled this - because there was an error with deploying to netlify from PRs that weren't mine. Will look into it

fix deployments

maybe fix

maybe fix
@BradyAJohnston
Copy link
Owner

OK for security reasons we can't have a preview deployment unless the PR is from me - but the docs now at least still build & report if successful.

@BradyAJohnston
Copy link
Owner

no rush on this - but I will be giving a tutorial to some CryoEM and CryoET people in a couple of weeks and would love to have a written tutorial for the starfile importing :)

@jojoelfe
Copy link
Contributor Author

jojoelfe commented May 7, 2024

Hey, I am slowly coming back to things after traveling.

I think the tutorial for cryo-EM SPA is in fairly good shape. It would be great to have something similar for cryo-ET, but there the problem is to find good datasets. An new choice would be some data from the CZI portal (https://cryoetdataportal.czscience.com/), but they don't use starfiles, but a json format. Maybe thats trivial to implement, but I haven't looked at it carefully yet.

@BradyAJohnston
Copy link
Owner

Just put together an initial .ndjson parser in #493 if you wanted to take a look. If you can point out some good datasets to use as testing on that portal (I've had a look, but this isn't my field of expertise).

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.

None yet

2 participants