Skip to content

Conversation

@aleixpuigb
Copy link
Collaborator

Fixes #400
Creates new term 'level of galvanic skin response'

@aleixpuigb aleixpuigb self-assigned this Oct 22, 2025
@aleixpuigb aleixpuigb requested a review from Copilot October 22, 2025 13:50
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR creates a new OBA term for "level of galvanic skin response" (OBA:2090004) to address issue #400. The term follows the entity_attribute DOSDP pattern and is defined as the amount of galvanic skin response.

Key changes:

  • Added new term OBA:2090004 with appropriate axioms and annotations
  • Imported NCIT:C93039 (Galvanic Skin Response) to support the new term
  • Updated version information from 2025-10-07 to 2025-10-22

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/patterns/definitions.owl Adds the new OBA:2090004 term definition with class declarations, annotations, equivalent classes axiom, and necessary imports; also updates ontology version date
src/patterns/data/default/entity_attribute.tsv Adds the DOSDP pattern template entry for OBA:2090004 linking galvanic skin response to the amount quality
Comments suppressed due to low confidence (1)

remove --term PR:000000001 \
--term SO:0000252 \
--term SO:0000234 \
--term SO:0000253 \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As this SO term and the CHEBI term CHEBI:17843 ('transfer RNA') have each other as equivalent class, the CI was failing.
@matentzn is this the correct way to solve the issue?

Copy link
Contributor

Choose a reason for hiding this comment

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

I am fine with this solution! We might revisit in the far future.

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.

[NTR] 'galvanic skin response measurement'

3 participants