-
Notifications
You must be signed in to change notification settings - Fork 11
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
Release bugfix version: 3.2.1 #158
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is the crux of this PR: there's too many negatives floating around here. We fold the 'store_false' action and --no- flags together: they cancel each other out. We get a positively worded flag, and make sure to set the default to True. Finally, we assign the ToggleActionFlag type so that both options appear together in the help. Note, we lose some short flags as a result (I don't think this is a loss for clarity). What was -bb, --no-backbone Do not sample backbone using inverse kinematics. (default: True) is now: --[no-]backbone Sample backbone using inverse kinematics. (default: True) This applies to: * -ns, --no-scale → --[no-]scale * -nosub, --no-subtract → --[no-]subtract * -bb, --no-backbone → --[no-]backbone * -sa, --no-sample-angle → --[no-]sample-angle * -T, --no-threshold-selection → --[no-]threshold-selection * -Ts, --no-segment-threshold-selection → --[no-]segment-threshold-selection
… Consistency! --write_intermediate_conformers → --write-intermediate-conformers --resolution_min → --resolution-min --clash_scaling_factor → --clash-scaling-factor --external_clash → --external-clash --bulk_solvent_level → --bulk-solvent-level --rmsd_cutoff → --rmsd-cutoff Dest variables will still maintain underscores.
The logic has to change here, since the variable was a negative.
Flags that changed: * --no-waters → --no-waters-clash Loss of short flags (now-required long form): * -ns (--no-scale) * -nosub (--no-subtract) * -nw (--no-waters-clash) * -bb (--no-backbone) * -sa (--no-sample-angle) * -T (--no-threshold-selection) Now-hyphenated flags: * --resolution_min → --resolution-min * --clash_scaling_factor → --clash-scaling-factor * --external_clash → --external-clash * --bulk_solvent_level → --bulk-solvent-level * --rmsd_cutoff → --rmsd-cutoff New flags: * --write-intermediate-conformers Also: * Remove unused import 'from os import path'
Flags that changed (maintained short form): * --build-stepsize → --dofs-per-iteration (-b) * --stepsize → --dihedral-stepsize (-s) * --no-waters → --no-waters-clash Loss of short flags (now-required long form): * -ns (--no-scale) * -nosub (--no-subtract) * -nw (--no-waters-clash) * -nb (--no-build) * -nl (--no-local) * -T (--no-threshold-selection) Now-hyphenated flags: * --resolution_min → --resolution-min * --clash_scaling_factor → --clash-scaling-factor * --external_clash → --external-clash * --bulk_solvent_level → --bulk-solvent-level * --write_intermediate_conformers → --write-intermediate-conformers
…-in-options Improve flags (& --help) of qfit programs
Described in the paper... functionality had disappeared...
Mechanism was there, needed to add the option into the ArgParser.
QFitSegment pushes its options.random_seed into the relabeller options.
Add --random-seed to provide option for reproducible runs
PDB: 7A4M; ApoFerritin Chain A
0.01=> self.options.rmsd_cutoff (imported via qFit Base options).
Add --rmsd-cutoff option to qFit-ligand
--write-intermediate-conformers => --write_intermediate_conformers
Update setup-miniconda GitHub Action (repairs failing CI)
adding CA option in the B-factor script & changing to remove b-factors from H atoms.
removing ::set-env flag and adding in environmental.yml to propagate to all tests.
adding file to create qfit conda environment
Add new analysis scripts, refactor get_metrics
…ction Correcting README example for post qFit refine cryoEM
1. Previous indentation meant we only looped through residues of the last chain. We now loop through residues for all matching chains. 2. Previously, only the last-calculated chain of rmsd data was stored. We now store RMSD for all matching chains. 3. Use an f-string filename for readability.
Fixup indentation in alpha_rmsd
Preserve SCALE/CRYST1 records through qfit_protein runs
Merge branch 'feature/vscode-development' into dev
…a-forge Switch to conda-forge: - Updated numpy requirements, and on Windows _only_, solving environment.yml was impossible. This is the main reason for the switch. - cvxopt hasn't been updated in a while in anaconda repo. Use the appropriate way to choose blas compiled against mkl, see https://conda-forge.org/docs/maintainer/knowledge_base.html#blas
Previously, this caused > FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result. Also: this returns a copy, not a view, but we will still copy just to be sure.
Move to python 3.7, 3.8
The LICENSE file exists for a reason, it is redundant to write the licence on top of every file.
Remove licences, update citation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release Notes
Users:
qfit_final_refine_xray.sh
orqfit_final_refine_cryoem.sh
--[no-]sample-angle
)--write_intermediate_conformers
between qfit_ligand and qfit_proteinDevelopment: