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

Fix typo in example workflow config and other cleanup #3480

Merged
merged 2 commits into from
Oct 2, 2020

Conversation

cdcapano
Copy link
Contributor

@cdcapano cdcapano commented Oct 2, 2020

Fixes a typo in the example workflow config files caught by Alex Brown. Source frame masses should be multiplied by the redshift to get detector frame mass, not divided. Also adds --no-create-proxy to the example pycbc_submit_dax command (that should of been there in the first place). Finally, this removes old workflow config files that are out of date and no longer used anywhere.

Copy link
Contributor

@sum33it sum33it left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this @cdcapano! It looks good to me.

@cdcapano cdcapano merged commit dd757c7 into gwastro:master Oct 2, 2020
@cdcapano cdcapano deleted the fix_infworkflowdocs branch October 2, 2020 14:31
ArthurTolley pushed a commit to ArthurTolley/pycbc that referenced this pull request Nov 14, 2022
* fix typo in workflow config, add no-create-proxy to submit dax command

* remove old inference workflow files
OliverEdy pushed a commit to OliverEdy/pycbc that referenced this pull request Apr 3, 2023
* fix typo in workflow config, add no-create-proxy to submit dax command

* remove old inference workflow files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants