Skip to content

Conversation

@agnesbrnb
Copy link
Collaborator

  • Script to extract tools from curated workflows
  • Upload a first JSON with tools extract from the 62 workflows kept after manual curation. Since tools name could be extracted from WorkflowHub steps description, the JSON file will need some curation.

Copy link
Contributor

@bebatut bebatut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @agnesbrnb. Looks great. I only put one comment

Comment on lines +179 to 185
matches_topic = set(self.edam_topic) & set(edam_keywords["topics"])
matches_operation = set(self.edam_operation) & set(edam_keywords["operations"])

if len(matches_topic) != 0 or len(matches_operation) != 0:
self.filtered_on = "edam"

return len(matches_topic) != 0 or len(matches_operation) != 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you create a has_edam_terms function in utils.py:

def has_edam_terms(edam_topics, edam_operations, edam_keywords):
   matches_topic = set(edam_topic) & set(edam_keywords["topics"])
   matches_operation = set(edam_operations) & set(edam_keywords["operations"])
   return len(matches_topic) != 0 or len(matches_operation) != 0

And change the function here as:

     has_edam_terms = utils.has_edam_terms(self.edam_topic, self.edam_operation, edam_keywords)
     if has_edam_terms:
        self.filtered_on = "edam"
     return has_edam_terms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants