search

ExcitedStates / qfit-3.0

qFit: Automated and unbiased multi-conformer models from X-ray and EM maps.
MIT License
34 stars 11 forks source link

removing options and running labeler by itself. #334

Closed stephaniewankowicz closed 1 year ago

stephaniewankowicz commented 1 year ago

This is now built into qFit protein

Pull Request Checklist

Cleaning up this script as we can run only relabler with qFit protein --only_segment option.

blake-riley commented 1 year ago

If you're gonna remove the option for users to run this as a standalone (reasonable), then check setup.py's entrypoints? I think there's a reference to it in there --- this will be dead if the main() function doesn't exist.

On Mon, May 1, 2023, 16:29 Stephanie (Mullane) Wankowicz < @.***> wrote:

This is now built into qFit protein Pull Request Checklist

  • Will your PR merge into the dev branch? Exceptions will be made for urgent bugfixes.
  • Have you forked from dev? If not, please rebase https://git-scm.com/book/en/v2/Git-Branching-Rebasing your PR onto @.***/git-rebase-and-git-rebase-onto-a6a3f83f9cce> the most recent dev tip.
  • Does your PR title succinctly describe the changes? Explain to a new user by completing the sentence: 'This PR will: ...'
  • Fill out the template below.

Cleaning up this script as we can run only relabler with qFit protein --only_segment option.

You can view, comment on, or merge this pull request online at:

https://github.com/ExcitedStates/qfit-3.0/pull/334 Commit Summary

File Changes

(1 file https://github.com/ExcitedStates/qfit-3.0/pull/334/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/ExcitedStates/qfit-3.0/pull/334, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABF3SZ7ETNZQK2KZL4UV4PTXEAMLVANCNFSM6AAAAAAXSEZYFI . You are receiving this because you are subscribed to this thread.Message ID: @.***>