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

Local point-by-point indexing notebook #238

Merged
merged 6 commits into from
Mar 4, 2024

Conversation

jadball
Copy link
Contributor

@jadball jadball commented Mar 4, 2024

  • Cleaned up and incorporated ImageD11/nbGui/run_pbp.ipynb as a scanning notebook: ImageD11/nbGui/S3DXRD/1_2_3_S3DXRD_pbp_indexing_local.ipynb
  • Added notebook to view the results: ImageD11/nbGui/S3DXRD/4_S3DXRD_view_PBP.ipynb
  • Many tasks still to do but it works, and it's fast!

@jonwright
Copy link
Member

Why is this deleting the run_pbp notebook?

@jonwright
Copy link
Member

I think I am missing a tool to view a notebook with diffs from git / github. The json is not readable at all. There is stuff in run_pbp that was correct ($HOME) that is a going to be a problem in the new notebook outside of the cluster (/home/esrf/$USER).

@jonwright
Copy link
Member

I will merge to keep a linear git history for now. Just tried to add an app called reviewnb to view the notebooks but didn't figure that out either.

@jonwright jonwright merged commit 811a659 into FABLE-3DXRD:master Mar 4, 2024
6 checks passed
@jadball
Copy link
Contributor Author

jadball commented Mar 4, 2024

@jonwright This just pulls the run_pbp notebook into the Scanning subdirectory with the rest of the notebooks and cleans up the paths to the dataset etc.

@jadball
Copy link
Contributor Author

jadball commented Mar 4, 2024

Problem with $HOME should be fixed in #240

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.

2 participants