MassBank / MassBank-web

The web server application and directly connected components for a MassBank web server
14 stars 22 forks source link

Exact mass display has far too many decimal places... #269

Closed schymane closed 3 years ago

schymane commented 3 years ago

Hi all,

It seems the exact mass display has gone a little crazy ... can we please truncate at a sensible number (e.g. the number of decimal places in the record ... or 4 d.p. ... but please not display like this?). Thanks!

image

image

image

https://github.com/MassBank/MassBank-data/blob/main/Eawag/EA000401.txt#L13

image

sneumann commented 3 years ago

Hi, I'd say fixed in the code, see https://msbi.ipb-halle.de/MassBank/RecordDisplay?id=EA000401 but not yet rolled out on MassBank.eu Yours, Steffen

tsufz commented 3 years ago

Hmm, as soon a committ makes its way to massbank.eu branch, the new server will be deployed. The preferred way is to push manually, could be also automated as I check the new deployed instance before going productive anyway.

Best, Tobias

meier-rene commented 3 years ago

I will take a look into it and report back.

meier-rene commented 3 years ago

I can not reproduce this issue. @tsufz have you already changed the codebase on massbank.eu? Its reporting 2.1.6 which is the latest release.

tsufz commented 3 years ago

I agree with @tsufz. Do you run your private MB server @schymane? Which is maybe not updated?

tsufz commented 3 years ago

Äh I agree with @meier-rene

tsufz commented 3 years ago

massbank.eu is fine: image

but massbank.jp is not: image

tsufz commented 3 years ago

Code base of massbank.jp is 2.1.4

schymane commented 3 years ago

Cannot reproduce any more either ... the best issue ever, fixed itself magically ;-)

tsufz commented 3 years ago

healing hands magics....