informatics-isi-edu / cirm-project

project repository for cirm. primarily for tracking tasks.
0 stars 0 forks source link

Age sorting is a little bit off #79

Open robes opened 7 years ago

robes commented 7 years ago

https://cirm-dev.isrd.isi.edu/chaise/recordset/#1/Microscopy:tissue/id=1/(term)=(Microscopy:Scan:tissue)@sort(age,id)@before(10%20weeks,3255)?limit=25

It starts off right:

svoinea commented 7 years ago

@robes The problem is related to recordset.

In the current search, I'm using an annotation like:

('Microscopy', 'Scan', 'age', 'description', '{"rank": "age_rank"}')

to specify the way to be sorted.

I think that the search page is sorting it properly.

robes commented 7 years ago

This will have to wait until we have a column sort annotation.

robes commented 7 years ago

We can now use this annotation

https://github.com/informatics-isi-edu/ermrest/blob/master/user-doc/annotation.md#2016-column-display

See 'column_order'

svoinea commented 7 years ago

@robes Maybe I misunderstood the way to use that annotation.

I have added the following annotation:

INSERT INTO _ermrest.model_column_annotation (schema_name, table_name, column_name, annotation_uri, annotation_value) VALUES
('Microscopy', 'Scan', 'age', 'tag:isrd.isi.edu,2016:column-display', 
'{
    "compact" :{"column_order":["age_rank"]}
}')
;

I got the error attached in the screenshot.

screenshot-age

svoinea commented 7 years ago

@robes I tried to interchange the age and age_rank columns in the above annotation, but that acted like not having the column_order annotation.

robes commented 7 years ago

we should find out why the error 409 is being generate, meaning why is the comparison generated incorrectly.