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

RNA Bases are filtered wrong #3174

Closed
SashaGraves opened this issue Aug 22, 2023 · 0 comments · Fixed by #3272
Closed

RNA Bases are filtered wrong #3174

SashaGraves opened this issue Aug 22, 2023 · 0 comments · Fixed by #3272
Assignees
@vitaepam
Labels
bug Macromolecules
Milestone
Macromolecules Mi...

Comments

@SashaGraves
Copy link
Contributor

SashaGraves commented Aug 22, 2023 •
edited
Loading

  1. Macromolecule builder
  2. Go to RNA library
  3. Go to Bases
  4. Scroll to G section (third, gray color)
  5. Choose second base (second G)

Observe:
This is Glycine, MonomerType: Peptide.

Investigate the behavior. Similar can be observed with other Rna Bases

image
@SashaGraves SashaGraves changed the title RNA Bases are filtered wrong Bug report - RNA Bases are filtered wrong Aug 22, 2023
@Nitvex Nitvex added the bug label Sep 6, 2023
@Nitvex Nitvex changed the title Bug report - RNA Bases are filtered wrong RNA Bases are filtered wrong Sep 6, 2023
vitaepam added a commit that referenced this issue Sep 7, 2023
@vitaepam
#3174 - Fix RNA Bases filtering
a4af34b
@vitaepam vitaepam mentioned this issue Sep 7, 2023
Merged
9 tasks
@olganaz olganaz mentioned this issue Sep 7, 2023
Closed
vitaepam added a commit that referenced this issue Sep 11, 2023
@vitaepam
#3174 - Update screenshots
87b41d1
vitaepam added a commit that referenced this issue Sep 19, 2023
@vitaepam
#3174 - Update tests
f381cc8
@Nitvex Nitvex added this to the Macromolecules Milestone 1 milestone Oct 2, 2023
rrodionov91 pushed a commit that referenced this issue Oct 4, 2023
@vitaepam @rrodionov91
#3174 - Fix RNA Bases filtering
99aea17
rrodionov91 pushed a commit that referenced this issue Oct 4, 2023
@vitaepam @rrodionov91
#3174 - Update screenshots
67ef61c
rrodionov91 pushed a commit that referenced this issue Oct 4, 2023
@vitaepam @rrodionov91
#3174 - Update tests
72fcc80
Nitvex added a commit that referenced this issue Oct 4, 2023
@Nitvex
#3174 – added click action in test
a3ffef5
Nitvex added a commit that referenced this issue Oct 4, 2023
@Nitvex
#3174 – fixed tests to compare screenshots
d084aa3
Nitvex added a commit that referenced this issue Oct 4, 2023
@vitaepam @Nitvex
#3174 - RNA Bases are filtered wrong (#3272)
9334918 
* #3174 - Fix  RNA Bases filtering

* #3174 - Update screenshots

* #3174 - Update tests

* Updated failing test

* #3174 – added click action in test

* #3174 – fixed tests to compare screenshots

---------

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

@vitaepam vitaepam

Labels
bug Macromolecules
Projects
None yet
Milestone
Macromolecules Milestone 1
Development

Successfully merging a pull request may close this issue.

#3174 - RNA Bases are filtered wrong epam/ketcher
4 participants
@Nitvex @SashaGraves @Zhirnoff @vitaepam