search

openmindanesth / openmind

anesthetic database
Eclipse Public License 1.0
1 stars 1 forks source link

deleting empty "repos" from extract #90

Closed hlesaint closed 4 years ago

hlesaint commented 4 years ago

"repos" should only appear on the extract if there is actually a repository linked to it.

tgetgood commented 4 years ago

That's reasonable. Should the same also apply to "related" and "tags"?

hlesaint commented 4 years ago

Yes, I think this would help to identify extracts that are incomplete or need some embedding.

hlesaint commented 4 years ago

Thanks for the fix. One brief consideration: I see the point for keeping the features' respective positioning on the extract. However, being not familiar with the concept it looks somewhat "untidy", as they were just randomly distributed or like a lack of proper positioning.

What do you think about keeping them on the extract, but making them very light grey instead?