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.

Sign up for GitHub

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

Jump to bottom

Updates and sanity checks to tomcat script #17

Open
wants to merge 2 commits into
base: experimental
Choose a base branch
from
Open

Updates and sanity checks to tomcat script #17

wants to merge 2 commits into from

Conversation

dveni
Copy link
Contributor

@dveni dveni commented Jul 1, 2024

  • Added possibility of parsing scan positions from json files (key names seem to be more consistent across detectors)
  • Now recdir argument accepts multiple '%s' (see helpstring, useful when reconstructions go to ra directly)
  • Added choices to 'units', 'xx_sign' and 'zz_sign' parameters to ensure the CLI throws an error if these arguments are not used correctly.
  • Added 'valid_scans' parameter to specify a list of strings the parsed paths must contain to be stitched (any of them must be present). It is useful for debugging when there are many scans to stitch.
  • Commented out the addition of the string '/rec_16bit_Paganin' in L192 since the reconstructions are sometimes named 'rec_16bit_phase' and this can be specified using the 'recdir' parameter.
dveni added 2 commits July 1, 2024 15:42
@dveni
Changelog:
bc60c30 
- Added possibility of parsing scan positions from json files (key names seems to be more consistent across detectors)
- Now recdir argument accepts multiple '%s' (see helpstring, useful when reconstructions go to ra directly)
- Added choices to 'units', 'xx_sign' and 'zz_sign' parameters to ensure the CLI throws an error if these arguments are not used correctly.
- Added 'valid_scans' parameter to specify a list of strings that the parsed paths must contain in order to be stitched (any of them must be present). Useful for debugging when there are many scans to stitch.
- Commented out the addition of the string '/rec_16bit_Paganin' in L192 since the reconstructions are sometimes named 'rec_16bit_phase' and this can be specified using the 'recdir' parameter.
@dveni
Remove list conversion
2129600
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@dveni