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

System should ignore spaces before line indexes on GenBank/GenPept sequences import #2763

Closed
AlexeyGirin opened this issue Feb 12, 2025 · 0 comments · Fixed by #2769 or #2770
Closed
Assignees
Labels
Export to Sequence Bucket: Bugs related to Export to Sequence mode Priority: High Severity: High High level visibility to the user

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Go to Macro - Sequence mode <--- Important
  2. Load following Sequence: GenBank-example.zip
    Image
    It is taken from https://www.ncbi.nlm.nih.gov/genbank/samplerecord/

Actual behavior
System throws exception: Convert error! Given string could not be loaded as (query or plain) molecule or reaction, see the error messages: 'SEQUENCE loader: Invalid symbols in the sequence: 1,6,1,1,2,1,1,8,1,2,4,1,3,0,1,3,6,1,4,2,1,4,8,1,5,4,1,6,0,1,6,6,1,7,2,1,7,8,1,8,4,1,9,0,1,9,6,1,1,0,2,1,1,0,8,1,1,1,4,1,1,2,0,1,1,2,6,1,1,3,2,1,1,3,8,1,1,4,4,1,1,5,0,1,1,5,6,1,1,6,2,1,1,6,8,1,1,7,4,1,1,8,0,1,1,8,6,1,1,9,2,1,1,9,8,1,2,0,4,1,2,1,0,1,2,1,6,1,2,2,2,1,2,2,8,1,2,3,4,1,2,4,0,1,2,4,6,1,2,5,2,1,2,5,8,1,2,6,4,1,2,7,0,1,2,7,6,1,2,8,2,1,2,8,8,1,2,9,4,1,3,0,0,1,3,0,6,1,3,1,2,1,3,1,8,1,3,2,4,1,3,3,0,1,3,3,6,1,3,4,2,1,3,4,8,1,3,5,4,1,3,6,0,1,3,6,6,1,3,7,2,1,3,7,8,1,3,8,4,1,3,9,0,1,3,9,6,1,4,0,2,1,4,0,8,1,4,1,4,1,4,2,0,1,4,2,6,1,4,3,2,1,4,3,8,1,4,4,4,1,4,5,0,1,4,5,6,1,4,6,2,1,4,6,8,1,4,7,4,1,4,8,0,1,4,8,6,1,4,9,2,1,4,9,8,1'
Image

Expected behavior
No exception, contend loaded
Image

Versions

  • Ketcher Version 3.1.0-rc.3 Build at 2025-02-11; 20:29:49
  • Indigo Version 1.29.0-rc.1.0-g2e30df8f7-wasm32-wasm-clang-19.0.0
  • Chrome Version 133.0.6943.60 (Official Build) (64-bit)
  • Win10

Issue found while testing - #1844

@AlexeyGirin AlexeyGirin added Export to Sequence Bucket: Bugs related to Export to Sequence mode Priority: High Severity: High High level visibility to the user labels Feb 12, 2025
@AlexeyGirin AlexeyGirin added this to the Indigo-1.29.0-rc.2 milestone Feb 12, 2025
@AliaksandrDziarkach AliaksandrDziarkach self-assigned this Feb 14, 2025
AliaksandrDziarkach added a commit that referenced this issue Feb 14, 2025
AliaksandrDziarkach added a commit that referenced this issue Feb 14, 2025
… GenBank/GenPept sequences import

backmerge to master
AlexanderSavelyev pushed a commit that referenced this issue Feb 17, 2025
AlexanderSavelyev pushed a commit that referenced this issue Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment