geneontology / noctua-form

2 stars 0 forks source link

Reimplanting table to ensure all data is read from Minerva #232

Closed tmushayahama closed 7 months ago

tmushayahama commented 9 months ago

We have successfully implemented a form for creating activities using a SHeX-like data structure. This dynamic form was essential due to the vast number of combinations of relationships and objects, making hard coding impractical.

However, we've encountered an issue with the table reading. The table reading was hard-coded, causing it to miss a significant amount of data fetched from Minerva. Specifically, if a relationship exists in SHeX but isn't recognized by the form (e.g., results_in_development), the data is not displayed correctly.

Steps to Reproduce:

Expected Result: The table should dynamically display all data fetched from Minerva based on the SHeX structure without any omissions.

Actual Result: The table misses out on certain data points from Minerva due to its hard-coded reading mechanism.

Recommendation: Reimplement the table reading steps to ensure it dynamically adapts to the SHeX data structure and accurately represents all data fetched from Minerva.

kltm commented 9 months ago

@tmushayahama @vanaukenk Just to clarify, is this part of a current project, or a proposal for something else?

tmushayahama commented 9 months ago

@kltm current project, existing bug

vanaukenk commented 9 months ago

Yes, this work is within the scope of the current project and addresses issues like: https://github.com/geneontology/noctua-form/issues/203

I've linked this ticket to the project.

tmushayahama commented 9 months ago

@vanaukenk @ukemi I think I have finished this one, I will go ahead and PR, so you can briefly take a look before workbench call. I think the table on ART can stay for now, that will bring lots of things. in other words if someone puts an unregistered edge results_in_development GO:term, they will not edit it on ART

vanaukenk commented 9 months ago

For testing:

  1. Check existing models to make sure that what is visible in the graph editor (i.e. what data exists for a model) is reflected in the Noctua form table
vanaukenk commented 9 months ago

I've made a test model for viewing relations display on the Noctua form table:

http://noctua-dev.berkeleybop.org/workbench/noctua-form/?model_id=gomodel%3A64f8fbd000000030

The table view looks great to me based on this test model.

@ukemi @LiNiMGI

ukemi commented 9 months ago

Looks good! I think the indentation/nesting is clear.

LiNiMGI commented 9 months ago

Looks good! I also notice: after adding a cc only annotation to the test model, the line says "GP info not available"...

image