VSCoding-Sequence icon indicating copy to clipboard operation
VSCoding-Sequence copied to clipboard

Launch Protein Viewer from File failed

Open zirui opened this issue 2 years ago • 6 comments

"Start Protein Viewer' with PDB works, but when select 'Launch Protein Viewer from Files(s)', vscode throw error msg: "Command 'Launch Protein Viewer from File(s)' resulted in an error (Cannot read properties of undefined (reading 'map'))

vscode version:

Version: 1.67.1 (Universal)
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:37:16.526Z (4 days ago)
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: 9.8.177.13-electron.0
OS: Darwin x64 20.6.0

zirui avatar May 11 '22 11:05 zirui

Hmm. I'm struggling to reproduce this @zirui. What sort of files are you trying to open? Maybe you can share them?

a-r-j avatar May 11 '22 12:05 a-r-j

i'm trying to open a pdb file(download from rcsb.org), but i think this error was not caused by the file itself, because it occurs before open files(when select 'Launch Protein Viewer from Files' in command palette)

zirui avatar May 13 '22 02:05 zirui

Ah, gotcha. You shouldn’t use the command palette to open from files. If you right click on the file(s) in the file explorer & select ‘Launch Protein Viewer from Files’ it should do the trick. Check out the GIF in the readme for an example :)

a-r-j avatar May 13 '22 02:05 a-r-j

@zirui How did you get on? Can I close the issue?

a-r-j avatar May 13 '22 23:05 a-r-j

right click on the file in the file explorer & select 'Launch Protein Viewer from Files' works! but i think it will be more convenient if view pdb from command palette is supported

zirui avatar May 16 '22 01:05 zirui

I agree that would be a nice feature. Unfortunately, I don’t think I’m going to have bandwidth to add it in the foreseeable future. Apologies.

Happy to support if you want to contribute a PR.

On 16 May 2022, at 02:54, zirui @.***> wrote:

 right click on the file in the file explorer & select 'Launch Protein Viewer from Files' works! but i think it will be more convenient if view pdb from command palette is supported

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

a-r-j avatar May 16 '22 12:05 a-r-j