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

Preview for ambiguous monomers #4555

Closed
olganaz opened this issue Apr 30, 2024 · 1 comment · Fixed by #5411
Closed

Preview for ambiguous monomers #4555

olganaz opened this issue Apr 30, 2024 · 1 comment · Fixed by #5411

Comments

@olganaz
Copy link
Collaborator

olganaz commented Apr 30, 2024

Background
Ambiguous monomers are used to describe ambiguity in the structure of a macromolecule.
Ambiguous monomers could be classified in two types:

  • Alternatives (XOR) - this type of ambiguous monomer is used to describe that at a specific position in the macromolecule, any monomer from a list of options can be present.
  • Mixed (AND) - this type of ambiguous monomer is used to specify not only the list of possible monomers that can be used, but also the probabilities (weights) with which these ambiguous may occur at that position.

The scope of this task includes preview for ambiguous monomers (chem structures, ratios, AND/XOR)

Requirements

  1. For the ambiguous monomer preview, it should contain the list of monomers (full names) that make up the ambiguous monomer.
  2. If the ambiguous monomer is "alternatives" (without probabilities) the list should be ordered alphabetically.
  3. If the ambiguous monomer is "mixed" (with probabilities) the list should be ordered by probabilities.
    3.1. If there are multiple monomers with same probability, they should be ordered alphabetically.
  4. The list should be limited to 5 monomers.
  5. For mixtures, the weights of each monomer should also be displayed in the preview (see mockups).
  6. In the case of X, "Any Amino Acid" should be displayed in the preview.
  7. In the case of N (both RNA and DNA versions), "Any Base" should be displayed in the preview.
  8. All the previews (micro mode, flex/snake mode, library) are as described, while the preview in sequence mode for nucleotides/nucleosides with a ambiguous base should have the name written as: sugar(base)phosphate (see mockups).

UX
image

@olganaz olganaz added the draft label Jun 29, 2024
@olganaz olganaz changed the title Macro: Preview for variant monomers Preview for variant monomers Jun 29, 2024
@rrodionov91 rrodionov91 added this to the Ketcher 2.25.0-rc.1 milestone Jul 22, 2024
@Zhirnoff Zhirnoff added the Test Cases Written Test cases has been written for that issue label Aug 12, 2024
@svvald svvald assigned svvald and unassigned Andrey-Antonov and olganaz Aug 21, 2024
@ljubica-milovic ljubica-milovic changed the title Preview for variant monomers Preview for ambiguous monomers Aug 26, 2024
@github-project-automation github-project-automation bot moved this to In progress in Ketcher Aug 27, 2024
@svvald svvald linked a pull request Aug 29, 2024 that will close this issue
9 tasks
@svvald svvald moved this from In progress to Code review in Ketcher Aug 29, 2024
@rrodionov91 rrodionov91 linked a pull request Sep 1, 2024 that will close this issue
9 tasks
@rrodionov91 rrodionov91 linked a pull request Sep 2, 2024 that will close this issue
9 tasks
rrodionov91 added a commit that referenced this issue Sep 3, 2024
- added preview for ambiguous monomers in sequence mode
- fixed preview position

---------

Co-authored-by: Nikita Chistousov <nikita_chistousov@epam.com>
@github-project-automation github-project-automation bot moved this from Code review to Ready for testing in Ketcher Sep 3, 2024
@AlexeyGirin
Copy link
Collaborator

Verified.
image

Versions

  • Ketcher Version 2.25.0-rc.1 Build at 2024-09-03; 15:31:03
  • Indigo Toolkit Version 1.24.0-rc.1.0-gb8f099851-wasm32-wasm-clang-19.0.0
  • Chrome Version 128.0.6613.114 (Official Build) (64-bit)
  • Win10

Guch1g0v pushed a commit that referenced this issue Oct 17, 2024
- added preview for ambiguous monomers in sequence mode
- fixed preview position

---------

Co-authored-by: Nikita Chistousov <nikita_chistousov@epam.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment