chembl / GLaDOS

Web Interface for ChEMBL @ EMBL-EBI
https://www.ebi.ac.uk/chembl/
Other
43 stars 5 forks source link

Things to amend - Compund Report Card #1328

Open nclopezo opened 6 months ago

nclopezo commented 6 months ago

Hi David, Thanks for the suggestions. They look good. :slightly_smiling_face: My comments are: • For the ‘Representations’: Please can you partially hide the molfile. It is very helpful to have the ‘copy’ button so easily available - but I really don’t think that anyone will want to see individual lines of the molfile within the web interface.

Slack Message

nclopezo commented 6 months ago

@FionaEBI also commented:

nclopezo commented 6 months ago

Hi Fiona,

nclopezo commented 6 months ago

Regarding the molfile, I created a new layout:

Screenshot 2023-12-11 at 14 16 30

https://wwwdev.ebi.ac.uk/chembl/compound_report_card/CHEMBL941/

It is on wwwdev now.

FionaEBI commented 6 months ago

re singular/plural: this is a bit of a muddle across the whole of chembl in both the database and the interface. In grammatically correct English, if a noun is "countable" like you can count multiple apples, then it can have an "s" but otherwise should not have an "s". However, just to make is more confusing, the American English system usually ignores this rule, so they would write "informations" while we would write "information" or "pieces of information", and similarly with "representation" - should be singular because it is not countable (not representations).

So the best practice guidelines for the chembl is to use the singular form for all tables/database fields in the database (but be aware that there is a real muddle in the database table names whereby this best practice rule has often not been followed). e.g. we have "chembl.compound_properties" but also "chembl.molecule_hierarchy", and also for database fields: "molecule_synonyms.synonyms" as well as "compound.records.compound_name" etc.

As a result, I think that the interface should also try to follow a similar format, and try to write all headings in the singular. But I agree that this there are many cases where either option would work. Happy to discuss further, no prob.

And I note, on reflection, it should probably be "chemical representation" rather than just "representation" for the heading that includes the molfile.

FionaEBI commented 6 months ago

Regarding the molfile, I created a new layout: Screenshot 2023-12-11 at 14 16 30

https://wwwdev.ebi.ac.uk/chembl/compound_report_card/CHEMBL941/

It is on wwwdev now.

This looks great now on dev - thanks!