communictation with pdb down

[grandrea]

Hello,
I tried to upload pdb via code today and got this error

[colin-combe]

mmtf.rcsb.org seems to be down

let's give it a bit to see if it comes back...

[colin-combe]

thanks for error report

[colin-combe]

ok... guess its not coming back https://www.rcsb.org/news/feature/663129f765d8a8cacaf76f9f

[colin-combe]

so i'm gonna have to fix this

[colin-combe]

its an NGL problem nglviewer/ngl#1045

@grandrea - you remember this thing whereby i hacked NGL so that chainnames could be 2 characters instead of 1? (colin-combe/ngl@71873ed)

How important is that still? It did break some things in NGL, just not things we used much.

In other words, what if I just updated to most recent version of NGL (without the hack) to fix this? Do you think that would mess things up for people?

[colin-combe]

it should be that this is fixed in the latest version of NGL on npm (https://www.npmjs.com/package/ngl).

So i should be able to fix this in xiVIEW and tidy some stuff up by importing NGL straight from npm (which xiVIEW already uses for managing JS dependencies).

But then you lose that 2 char hack.

Awaiting your input.

[grandrea]

bye bye 2 char hack.

[grandrea]

(that work is done afaik)

[colin-combe]

So i should be able to fix this in xiVIEW and tidy some stuff up by importing NGL straight from npm

good that we have tests, because they now fail (after updating to most recent NGL).

Will look into what the problems are...

[colin-combe]

Things working again now?

[grandrea]

yes, all good