GLaDOS icon indicating copy to clipboard operation
GLaDOS copied to clipboard

Things to amend - June 2023

Open nclopezo opened this issue 1 year ago • 6 comments

  • [x] 1. Add a custom sorting for the release date filter in documents.
  • [ ] 2. Remove time from release date. Screenshot 2023-11-01 at 09 31 57
  • [x] 3. Show Molecule Features for all the compounds
  • [x] 4. Delete text 'Synthetic' from 'Synthetic Small Molecule'
  • [x] 5. Add new icon for chemical probe.
  • [x] 6. Add new icon for natural product.
  • [x] 7. Fix the overlapping of texts in the histogram bars: Screenshot 2023-07-04 at 14 06 44

nclopezo avatar Jun 26 '23 14:06 nclopezo

@juanfmx2 I checked and regarding 2 it would be better if we had a text from elasticsearch without the time part ('T00:00:00'), because it will be fixed from the source, instead to having to apply the parsing everywhere it appears on the interface.

nclopezo avatar Jun 26 '23 14:06 nclopezo

Regarding task 1, now we can set up a custom configuration for certain properties. For example, for ChEMBL Release in documents, now you can select up to 100 bars, this means that you can see all the releases in the histogram:

https://github.com/chembl/GLaDOS/assets/1944922/1864824b-7bf4-494c-95ec-c0faf4125b98

Now you can also sort by either key or count, this means that you can sort by the release name instead of the number of items of the bar:

https://github.com/chembl/GLaDOS/assets/1944922/24a51ce4-3c5d-416a-ac9e-6e48d04e772f

As it was expected, the order is lexicographic. I tried to add a custom script to sort it in a 'natural' way, but it was not possible, and we will have the same problem when sorting on the table. I think is better to make a field with a name like 'release_label' that has the number first, and the values are something like:

  • '1. CHEMBL_1'
  • '2. CHEMBL_2'
  • '3. CHEMBL_3'
  • '4. CHEMBL_4'

That way we can sort it in a natural way easily.

nclopezo avatar Jul 04 '23 15:07 nclopezo

I think even if the number is leading it will not solve the sorting issue, because is will still be a string the only way to solve it is to add leading zeros to the the releases with a single digit (CHEMBL_01, CHEMBL_02, CHEMBL_03 . . . CHEMBL_09)

juanfmx2 avatar Jul 04 '23 16:07 juanfmx2

Yes, you are right, my bad, the way to fix it would be to add leading zeros.

nclopezo avatar Jul 05 '23 08:07 nclopezo

There is a new custom sorting for the ChEMBL release filter: Screenshot 2023-11-01 at 09 34 19

More releases appear as the number of bars is increased. Screenshot 2023-11-01 at 09 35 34

nclopezo avatar Nov 01 '23 09:11 nclopezo

It looks great now, thank you!

BZdrazil avatar Nov 01 '23 12:11 BZdrazil